teksilo_core/widget_tree/

drag_drop_impl.rs

// SPDX-License-Identifier: MPL-2.0
// SPDX-FileCopyrightText: 2026 FernTech

use super::*;

// App-global stash for the typed payload of an in-flight app-originated OS
// drag. When an in-app drag escalates to a native OS drag at the window
// boundary, the OS only carries the serialized MIME bytes — the typed
// `Box<dyn Any>` fast-path value would be lost. We park the whole `DragPayload`
// here for the lifetime of the OS drag so that if the drag wanders back over
// **any** window of this app (the source window or another one), that window
// can recover the original typed payload and present it as a normal internal
// drag. Single-threaded GUI ⇒ a thread-local is the process-wide registry, and
// it is only ever touched on the main thread (escalation and all
// `*_external_drag` / `handle_os_drag_ended` routing run there; the Wayland
// dispatch thread only `post_external`s).
//
// **Liveness gate.** Recovery is gated on the `live` flag, not on payload
// presence. `live` is set true by `outbound_begin` (escalation) and cleared by
// `outbound_end` (the terminal `DragEnded`, or a source-window close). This is
// what keeps a leaked / stale payload from misclaiming a *later* genuine
// external drag from another application: `outbound_take_if_live` only hands
// the payload back while `live`, and `outbound_restash` is a no-op once the
// drag has ended — so even a racing re-stash (cross-window drop-on-nothing)
// can't resurrect a finished drag.
struct OutboundStash {
    /// True while an app-originated OS drag is in flight.
    live: bool,
    /// The parked typed payload, present whenever no window currently holds it
    /// as a re-entered session.
    payload: Option<crate::drag_payload::DragPayload>,
    /// The pointer that started the drag.
    ///
    /// Parked beside the payload because **no OS drag protocol names the
    /// dragging device to the destination**: `wl_data_device`, XDND, OLE
    /// `IDropTarget` and `NSDraggingDestination` all describe the data and the
    /// position and say nothing about the hand. So for a drag from another
    /// application the kind is genuinely unknown — but for our OWN drag
    /// re-entering one of our windows it is not, and this is where the window
    /// that recovers the payload also recovers the device, which is what makes
    /// the re-entered session's hover slop, drop bands and preview clearance
    /// read the finger rather than a mouse.
    pointer: Option<crate::pointer::PointerInfo>,
}

thread_local! {
    static OUTBOUND: std::cell::RefCell<OutboundStash> = const {
        std::cell::RefCell::new(OutboundStash {
            live: false,
            payload: None,
            pointer: None,
        })
    };
}

/// Begin an outbound drag: mark live and park the typed payload plus the
/// pointer that is carrying it.
fn outbound_begin(payload: crate::drag_payload::DragPayload, pointer: crate::pointer::PointerInfo) {
    OUTBOUND.with(|s| {
        let mut s = s.borrow_mut();
        s.live = true;
        s.payload = Some(payload);
        s.pointer = Some(pointer);
    });
}

/// Recover the parked payload **only while the drag is live**. Leaves `live`
/// set (a window now holds the payload as a re-entered session).
fn outbound_take_if_live() -> Option<crate::drag_payload::DragPayload> {
    OUTBOUND.with(|s| {
        let mut s = s.borrow_mut();
        if s.live { s.payload.take() } else { None }
    })
}

/// The pointer of the in-flight outbound drag, while it is live. Read by the
/// window a re-entered drag lands in, so its session names the real device.
fn outbound_pointer_if_live() -> Option<crate::pointer::PointerInfo> {
    OUTBOUND.with(|s| {
        let s = s.borrow();
        if s.live { s.pointer } else { None }
    })
}

/// Whether an app-originated OS drag is still in flight. A window holding a
/// re-entered session whose stash has gone reads it to notice that the drag it
/// is showing feedback for has ended elsewhere.
fn outbound_is_live() -> bool {
    OUTBOUND.with(|s| s.borrow().live)
}

/// Return a re-entered payload to the stash so another window can recover it —
/// but only if the drag is still live (a racing terminal event may have ended
/// it first, in which case the payload is dropped).
fn outbound_restash(payload: crate::drag_payload::DragPayload) {
    OUTBOUND.with(|s| {
        let mut s = s.borrow_mut();
        if s.live {
            s.payload = Some(payload);
        }
    });
}

/// Whether a payload is currently parked. Test/diagnostic helper.
#[cfg(test)]
fn has_outbound_typed() -> bool {
    OUTBOUND.with(|s| s.borrow().payload.is_some())
}

/// End the outbound drag: clear the live flag and drop any parked payload.
/// Idempotent. After this, no window can recover the payload.
fn outbound_end() {
    OUTBOUND.with(|s| {
        let mut s = s.borrow_mut();
        s.live = false;
        s.payload = None;
        s.pointer = None;
    });
}

/// The pointer an **inbound OS drag from another application** is credited to.
///
/// `PointerKind::Unknown` is the truthful answer, not a hedge: no OS drag
/// protocol tells the destination which device the source is dragging with, and
/// `Unknown` reads as precise everywhere a kind is consulted — the same
/// behaviour every OS drop had before pointers were distinguishable. The id is
/// the mouse's because the drag is following the system cursor and because an
/// external session never enters the pointer table.
fn unknown_external_pointer() -> crate::pointer::PointerInfo {
    let mut info = crate::pointer::PointerInfo::mouse(crate::pointer::EventTime::ZERO);
    info.kind = teksilo_tokens::PointerKind::Unknown;
    info
}

/// Which half of the drag pipeline [`WidgetTree::drive_drag_session`] runs.
#[derive(Copy, Clone, PartialEq, Eq, Debug)]
enum DragDrive {
    /// Re-target and re-feedback at `position` (`handle_drag_move`).
    Move,
    /// Complete the drag at `position` (`handle_drag_drop`).
    Drop,
}

impl WidgetTree {
    /// Run one half of the drag pipeline with the **drag's own pointer**
    /// installed as the input snapshot.
    ///
    /// Every entry point that is not a pointer sample goes through here: an OS
    /// drag's phases (delivered from a platform thread) and the per-layout drag
    /// tick. Without it `current_input` holds its default — a mouse — so
    /// `on_drag_hover` / `on_drag_tick` / `on_drop` were told they were serving
    /// a mouse for the whole of a finger drag, and the hit test that picks the
    /// target used a mouse's slop. Saved and restored around the call the same
    /// way every dispatch site treats the snapshot, so a nested dispatch cannot
    /// leak it.
    fn drive_drag_session(
        &mut self,
        position: teksilo_canvas::Point,
        which: DragDrive,
        ops: &mut dyn crate::window::WindowOps,
    ) -> Option<crate::drag_payload::DropOutcome> {
        let pointer = self.active_drag.as_ref().map(|d| d.pointer)?;
        let saved = std::mem::replace(
            &mut self.current_input,
            crate::pointer::InputSnapshot::for_drag_session(pointer),
        );
        let outcome = match which {
            DragDrive::Move => {
                self.handle_drag_move(position, ops);
                None
            }
            DragDrive::Drop => self.handle_drag_drop(position, ops),
        };
        self.current_input = saved;
        outcome
    }

    /// Clean up drag preview overlay (if any).
    ///
    /// The preview content is a built widget subtree, so it goes through
    /// `destroy_subtree` like every other teardown: a bare `arena.destroy`
    /// would leave its animations, animated-quad slots, bindings,
    /// subscriptions and shortcuts registered until the tree drops.
    pub(super) fn cleanup_drag_preview(&mut self) {
        let Some((overlay_id, content_id)) = self
            .active_drag
            .as_ref()
            .map(|drag| (drag.preview_overlay_id, drag.preview_content_id))
        else {
            return;
        };
        if let Some(overlay_id) = overlay_id {
            self.overlay_manager.dismiss(overlay_id);
        }
        if let Some(content_id) = content_id {
            self.destroy_subtree(content_id);
        }
    }

    /// Cancel the active drag session: fire `on_drag_leave` on the current
    /// target (if any), dismiss the preview overlay, clear the session and
    /// release pointer capture. Used by Escape, explicit cancel requests,
    /// and the source-destroyed salvage in `revalidate_interaction_state`.
    pub(super) fn cancel_active_drag(&mut self, ops: &mut dyn crate::window::WindowOps) {
        // A native start asked for and not yet confirmed must not happen after
        // the drag it was for has ended here.
        if self.os_drag_pending.take().is_some() {
            ops.cancel_os_drag();
        }
        let prev_target = self.active_drag.as_ref().and_then(|d| d.current_target);
        // The source widget, so a cancelled in-app drag still notifies its
        // originator via `on_drag_ended(Cancelled)`. External drags carry no
        // source (`None`), so they never fire it.
        let source = self.active_drag.as_ref().and_then(|d| d.source_widget);
        // Serve the teardown as the drag's own pointer. Every route into here is
        // one where the current snapshot is about something else or nothing at
        // all — an Escape key, a layout-pass reap, a platform abort — and a
        // handler that branches on the device must get the same answer at the end
        // of a drag as it did in the middle of it. Saved and restored, so a
        // teardown reached from inside another dispatch leaves that dispatch's
        // snapshot as it found it.
        let saved = self.active_drag.as_ref().map(|d| d.pointer).map(|p| {
            std::mem::replace(
                &mut self.current_input,
                crate::pointer::InputSnapshot::for_drag_session(p),
            )
        });
        self.cleanup_drag_preview();
        self.active_drag = None;
        self.os_drop_accepted = None;
        self.release_drag_capture(source);
        self.current_cursor = crate::widget::CursorIcon::Default;
        if let Some(prev) = prev_target {
            self.fire_on_drag_leave(prev, &mut *ops);
        }
        if let Some(src) = source {
            self.fire_on_drag_ended(src, crate::drag_payload::DropOutcome::Cancelled, &mut *ops);
        }
        if let Some(saved) = saved {
            self.current_input = saved;
        }
    }

    // --- External (OS) drag-and-drop -----------------------------------
    //
    // OS drops (files / text / URLs dragged from another application or the
    // file manager) reuse the *entire* internal drag pipeline. Rather than a
    // parallel set of handlers, an external drag synthesises a `DragSession`
    // carrying a `DragPayload::external(...)` and then drives the same
    // `handle_drag_move` / `handle_drag_drop` / `cancel_active_drag` paths, so
    // any widget with `on_drag_hover` / `on_drag_leave` / `on_drop` works for
    // both internal and external drags. Widgets distinguish the source via
    // `payload.is_external()` / `payload.files()` etc.
    //
    // Differences from internal drags: there is no in-app source widget
    // (`source_widget = None`), no pointer capture (the OS owns the pointer
    // during its drag loop), and no in-tree preview overlay (the OS renders
    // its own drag image).

    /// Begin an external drag session at `position` carrying OS-delivered
    /// `data`. Establishes the initial hover target and feedback immediately.
    ///
    /// # Which device is dragging
    ///
    /// A drag from **another application** does not say: none of
    /// `wl_data_device`, XDND, OLE `IDropTarget` or `NSDraggingDestination`
    /// carries the source's device to the destination, so the session reports
    /// [`PointerKind::Unknown`](teksilo_tokens::PointerKind::Unknown) — which
    /// resolves as precise everywhere a kind is read, i.e. exactly the
    /// behaviour every OS drop had before pointers were distinguishable.
    ///
    /// Our OWN escalated drag re-entering a window is the case that *is*
    /// knowable, and it does not go through this door blind: the pointer is
    /// recovered from the outbound stash alongside the typed payload.
    pub fn begin_external_drag(
        &mut self,
        position: teksilo_canvas::Point,
        data: crate::drag_payload::ExternalDropData,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        // An inbound drag over the window that asked for a native start is that
        // native drag: while the pointer is held for our drag, no other
        // application's drag can reach us. The platform has the pointer, so
        // hand the in-app session over first, as its confirmation would have.
        if let Some(pending) = self.os_drag_pending {
            self.confirm_os_drag(pending, &mut *ops);
        }

        // Defensively clear any stale session (e.g. a re-entered drag that
        // never delivered a matching leave). cancel_active_drag fires
        // on_drag_leave on the previous target first.
        if self.active_drag.is_some() {
            self.cancel_active_drag(&mut *ops);
        }

        // Is this our own app's in-flight OS drag wandering (back) over a
        // window? A non-empty global stash means an app-originated OS drag is
        // live (only one OS drag exists at a time), so recover the original
        // typed payload and present it as a normal *internal* drag. In-app
        // targets then see the typed value — this is what enables a drag to
        // round-trip out and back, and drag-and-drop between two windows of the
        // same app. The terminal `on_drag_ended` is owned by the source window
        // (via `DragEnded`), so this re-entered session carries no
        // `source_widget` and never fires it on drop.
        if let Some(mut payload) = outbound_take_if_live() {
            // Also expose the file/text/URI view derived from the carried MIME,
            // so the re-entered drag satisfies external-style targets (DropZone)
            // in addition to typed in-app targets.
            payload.enrich_external_from_mime();
            // The device is recoverable here and nowhere else: this is our own
            // drag coming home, and the stash kept the pointer that armed it.
            let pointer = outbound_pointer_if_live().unwrap_or_else(unknown_external_pointer);
            self.active_drag = Some(crate::drag_state::DragSession {
                payload,
                pointer,
                source_widget: None,
                is_external: false,
                current_position: position,
                current_target: None,
                feedback: crate::drag_state::DropFeedback::NoFeedback,
                preview_content_id: None,
                preview_overlay_id: None,
            });
            self.os_drag_reentered = true;
            self.drive_drag_session(position, DragDrive::Move, &mut *ops);
            return;
        }

        self.active_drag = Some(crate::drag_state::DragSession {
            payload: crate::drag_payload::DragPayload::external(data),
            pointer: unknown_external_pointer(),
            source_widget: None,
            is_external: true,
            current_position: position,
            current_target: None,
            feedback: crate::drag_state::DropFeedback::NoFeedback,
            preview_content_id: None,
            preview_overlay_id: None,
        });
        // No pointer capture, no Grabbing cursor — the OS owns the drag image
        // and cursor during an external drag.
        self.drive_drag_session(position, DragDrive::Move, &mut *ops);
    }

    /// Update an in-flight external drag as the OS reports pointer motion.
    /// No-op unless an external session — or our own re-entered OS drag, which
    /// is an internal session — is active.
    pub fn update_external_drag(
        &mut self,
        position: teksilo_canvas::Point,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        // Drives both a genuine external drag and our own re-entered OS drag
        // (now an internal session). `handle_drag_move` re-stashes and re-exits
        // if a re-entered drag leaves the window again.
        if self.active_drag.as_ref().is_some_and(|d| d.is_external) || self.os_drag_reentered {
            self.drive_drag_session(position, DragDrive::Move, &mut *ops);
        }
    }

    /// Complete an external drag with a drop at `position`, firing `on_drop`
    /// on the target. `data` is the authoritative payload read at drop time;
    /// if non-empty it replaces the session payload (some backends only have
    /// the full data at drop, not at enter). No-op unless an external session
    /// — or our own re-entered OS drag, which is an internal session — is
    /// active.
    pub fn end_external_drag(
        &mut self,
        position: teksilo_canvas::Point,
        data: crate::drag_payload::ExternalDropData,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        // Our own OS drag dropped inside an app window: complete it as an
        // internal drop with the recovered typed payload. The re-entered
        // session has no `source_widget`, so `handle_drag_drop` fires `on_drop`
        // on the target but not `on_drag_ended` — the source window fires that
        // once when the OS posts the terminal `DragEnded`. Clear the global
        // stash so that trailing event treats the drag as finished.
        if self.os_drag_reentered {
            self.os_drag_reentered = false;
            let outcome = self.drive_drag_session(position, DragDrive::Drop, &mut *ops);
            // Dropped back on the window that started the drag: what the source
            // must hear is the in-app outcome, which the platform cannot know
            // (it reports a copy). Held until the platform's `DragEnded`, which
            // is what fires the source's `on_drag_ended`. A drop on another
            // window of the app keeps the platform's outcome.
            if self.outbound_drag_source.is_some()
                && let Some(in_app @ crate::drag_payload::DropOutcome::InApp { .. }) = outcome
            {
                self.reentered_drop_outcome = Some(in_app);
            }
            outbound_end();
            return;
        }
        if !self.active_drag.as_ref().is_some_and(|d| d.is_external) {
            return;
        }
        if !data.is_empty()
            && let Some(drag) = self.active_drag.as_mut()
        {
            drag.payload = crate::drag_payload::DragPayload::external(data);
        }
        self.drive_drag_session(position, DragDrive::Drop, &mut *ops);
    }

    /// End an external drag over this window **for good**: the OS aborted the
    /// operation, or the app-originated drag this window was holding as a
    /// re-entered session has finished elsewhere. No drop will follow.
    ///
    /// The difference from [`cancel_external_drag`](Self::cancel_external_drag)
    /// is the re-entered case, and it is the whole reason both exist: a leave
    /// *re-stashes* the typed payload so the next window the drag enters can
    /// pick it up, because the OS drag is still in flight. An abort must not —
    /// re-stashing a dead drag leaves a payload that the next genuine external
    /// drag from another application could misclaim.
    ///
    /// `on_drag_leave` fires on the current target so no highlight is stranded.
    /// `on_drag_ended` fires **only** for a session with an in-app source, so a
    /// re-entered session is silent here: the window that started the drag owns
    /// that notification and fires it once from
    /// [`handle_os_drag_ended`](Self::handle_os_drag_ended).
    pub fn abort_external_drag(&mut self, ops: &mut dyn crate::window::WindowOps) {
        if self.active_drag.is_none() {
            self.os_drag_reentered = false;
            return;
        }
        self.os_drag_reentered = false;
        self.cancel_active_drag(&mut *ops);
    }

    /// Cancel an in-flight external drag (the pointer left the window or the
    /// OS aborted the operation) without dropping. No-op unless an external
    /// session — or our own re-entered OS drag, which is an internal session
    /// that re-exits rather than cancels — is active.
    pub fn cancel_external_drag(&mut self, ops: &mut dyn crate::window::WindowOps) {
        // A re-entered OS drag leaving the window again must NOT cancel the
        // whole drag (the OS drag is still live) — re-stash the typed payload
        // for the next window it enters and tear down this internal session
        // without a terminal `on_drag_ended`.
        if self.os_drag_reentered {
            self.reexit_outbound(&mut *ops);
        } else if self.active_drag.as_ref().is_some_and(|d| d.is_external) {
            self.cancel_active_drag(&mut *ops);
        }
    }

    /// Fire `on_drag_tick` on the current drop target (if any). Runs once
    /// per layout pass while a drag session is active. The handler
    /// receives the pointer position in the target's local coordinates.
    /// Fires from both external and own handler buckets.
    pub(super) fn process_drag_tick(&mut self, ops: &mut dyn crate::window::WindowOps) {
        // First: a re-entered OS drag whose OS session has ended elsewhere.
        self.reap_dead_reentered_drag(&mut *ops);
        let Some((target_id, position, pointer)) = self
            .active_drag
            .as_ref()
            .and_then(|d| d.current_target.map(|t| (t, d.current_position, d.pointer)))
        else {
            return;
        };
        if !self.arena.is_active(target_id) {
            return;
        }
        let bounds = self.arena.bounds(target_id);
        let local = teksilo_canvas::Point::new(position.x - bounds.x, position.y - bounds.y);
        let (mut ext_handler, mut own_handler) = match self.arena.get_mut(target_id) {
            Some(node) => (
                node.external_handlers.on_drag_tick.take(),
                node.handlers.on_drag_tick.take(),
            ),
            None => return,
        };
        if ext_handler.is_none() && own_handler.is_none() {
            return;
        }
        // The tick fires from `layout()`, outside any sample, so the snapshot
        // has to be installed here or the handler is told it is serving a mouse
        // — which is what made the coarse auto-scroll band unreachable for the
        // whole of a finger drag.
        let saved = std::mem::replace(
            &mut self.current_input,
            crate::pointer::InputSnapshot::for_drag_session(pointer),
        );
        let mut ctx = self.make_event_context(&mut *ops);
        if let Some(h) = ext_handler.as_mut() {
            h(local, &mut ctx);
        }
        if let Some(h) = own_handler.as_mut() {
            h(local, &mut ctx);
        }
        if let Some(node) = self.arena.get_mut(target_id) {
            node.external_handlers.on_drag_tick = ext_handler;
            node.handlers.on_drag_tick = own_handler;
        }
        self.collect_from_ctx(ctx, target_id);
        // If the tick handler scrolled content, the pointer is now over a
        // different item — refresh the hover pipeline with the same
        // pointer position so feedback reflects the new content offset.
        if self.active_drag.is_some() {
            self.handle_drag_move(position, &mut *ops);
        }
        self.current_input = saved;
    }

    /// End a re-entered OS drag whose OS session has finished somewhere else.
    ///
    /// The terminal `DragEnded` is posted to the window that **started** the
    /// drag, and that window is not necessarily the one currently showing the
    /// re-entered session: drag a row out of window A, over window B, and let
    /// the compositor abort it, and B is left holding a live `active_drag`, a
    /// highlighted drop target and an `os_drag_reentered` flag for a drag that
    /// no longer exists — for the rest of the process, since the OS will send B
    /// nothing further. The outbound stash is process-wide and is cleared by
    /// the terminal event, so "I hold a re-entered session and the stash is
    /// dead" is the exact condition, and every window runs a layout pass.
    ///
    /// Deliberately *not* a fan-out from the terminal event: reaching every
    /// window's tree from the one being routed to needs the window manager, and
    /// the condition is already visible from inside each tree.
    fn reap_dead_reentered_drag(&mut self, ops: &mut dyn crate::window::WindowOps) {
        if !self.os_drag_reentered || outbound_is_live() {
            return;
        }
        crate::trace_input!(
            Gestures,
            "the OS drag this window held as re-entered has ended elsewhere: dropping the session"
        );
        self.abort_external_drag(&mut *ops);
    }

    /// Fire `on_drag_leave` on the given widget (if it has one), mark it
    /// needs_paint, and process any commands the handler emitted. Used
    /// whenever a drop target stops being the current target — whether
    /// because the pointer moved elsewhere, the drop completed, or the
    /// drag was cancelled. Fires from both external and own buckets.
    pub(super) fn fire_on_drag_leave(
        &mut self,
        target_id: WidgetId,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        if !self.arena.is_active(target_id) {
            return;
        }
        let (mut ext_handler, mut own_handler) = match self.arena.get_mut(target_id) {
            Some(node) => (
                node.external_handlers.on_drag_leave.take(),
                node.handlers.on_drag_leave.take(),
            ),
            None => return,
        };
        if ext_handler.is_none() && own_handler.is_none() {
            // Still mark for repaint so any visual artefacts the
            // framework owns (feedback lines, highlights) clear.
            self.arena.mark_needs_paint(target_id);
            return;
        }
        let mut ctx = self.make_event_context(&mut *ops);
        if let Some(h) = ext_handler.as_mut() {
            h(&mut ctx);
        }
        if let Some(h) = own_handler.as_mut() {
            h(&mut ctx);
        }
        if let Some(node) = self.arena.get_mut(target_id) {
            node.external_handlers.on_drag_leave = ext_handler;
            node.handlers.on_drag_leave = own_handler;
        }
        self.collect_from_ctx(ctx, target_id);
        self.arena.mark_needs_paint(target_id);
    }

    /// Fire `on_drag_ended` on a drag's **source** widget with the final
    /// outcome (in-app drop, OS export, or cancel). Mirrors
    /// [`Self::fire_on_drag_leave`]'s take/restore-handler discipline.
    pub(super) fn fire_on_drag_ended(
        &mut self,
        source_id: WidgetId,
        outcome: crate::drag_payload::DropOutcome,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        if !self.arena.is_active(source_id) {
            return;
        }
        // Handlers attached at the widget's creation site live in the
        // `external_handlers` bucket; those installed from the widget's own
        // `build()` live in `handlers`. Fire whichever is present (both, if
        // both) — same dual-bucket discipline as `fire_on_drag_leave`.
        let (mut ext_handler, mut own_handler) = match self.arena.get_mut(source_id) {
            Some(node) => (
                node.external_handlers.on_drag_ended.take(),
                node.handlers.on_drag_ended.take(),
            ),
            None => return,
        };
        if ext_handler.is_none() && own_handler.is_none() {
            return;
        }
        let mut ctx = self.make_event_context(&mut *ops);
        if let Some(h) = ext_handler.as_mut() {
            h(outcome, &mut ctx);
        }
        if let Some(h) = own_handler.as_mut() {
            h(outcome, &mut ctx);
        }
        if let Some(node) = self.arena.get_mut(source_id) {
            node.external_handlers.on_drag_ended = ext_handler;
            node.handlers.on_drag_ended = own_handler;
        }
        self.collect_from_ctx(ctx, source_id);
    }

    /// Window content size (logical px) from the last layout proposal, if both
    /// axes were exact. Used to detect when an in-app drag leaves the window.
    fn window_content_size(&self) -> Option<(f32, f32)> {
        Some((self.last_proposal.width?, self.last_proposal.height?))
    }

    /// Whether `position` is outside this window's content rect. Unknown bounds
    /// (non-exact proposal) ⇒ never treated as outside.
    fn is_outside_window(&self, position: teksilo_canvas::Point) -> bool {
        match self.window_content_size() {
            Some((w, h)) => {
                position.x < 0.0 || position.y < 0.0 || position.x > w || position.y > h
            }
            None => false,
        }
    }

    /// Hand an **internal** drag whose payload is OS-exportable to the platform
    /// as a native OS drag: when it leaves the window bounds, or, with
    /// [`OsHandoff::Eager`](crate::drag_payload::OsHandoff::Eager), on a real
    /// pointer sample (`sample`) anywhere. Returns `true` if it consumed the
    /// move (handed over, or re-exited a re-entered drag); `false` when the
    /// in-app flow continues, which includes a start the platform answered
    /// [`OsDragStart::Pending`](crate::drag_payload::OsDragStart::Pending).
    fn try_escalate_to_os_drag(
        &mut self,
        position: teksilo_canvas::Point,
        sample: bool,
        ops: &mut dyn crate::window::WindowOps,
    ) -> bool {
        // Already handed off to the OS and currently re-entered into this
        // window: leaving again must NOT start a second OS drag. Re-stash the
        // typed payload (so the next window can recover it) and tear the
        // internal session down without a terminal `on_drag_ended`.
        if self.os_drag_reentered {
            if self.is_outside_window(position) {
                self.reexit_outbound(&mut *ops);
                return true;
            }
            return false;
        }

        // A start is already in flight: the in-app drag carries on until the
        // platform confirms it, and must not ask a second time.
        if self.os_drag_pending.is_some() {
            return false;
        }

        // Only a plain internal drag with an exportable payload escalates.
        let (data, handoff, dragging) = match self.active_drag.as_ref() {
            Some(d)
                if !d.is_external && d.source_widget.is_some() && d.payload.is_os_exportable() =>
            {
                (d.payload.to_outbound(), d.payload.os_handoff(), d.pointer)
            }
            _ => return false,
        };
        // `Eager` asks only from a real pointer sample: macOS needs the live
        // drag event to start a native drag, and a drag tick runs outside any
        // sample. A tick leaves the next sample to ask.
        let eager = handoff == crate::drag_payload::OsHandoff::Eager && sample;
        if !eager && !self.is_outside_window(position) {
            return false;
        }

        // Ask the platform to start a native OS drag. A refusal (no backend /
        // test sink, a stale event on macOS) keeps the in-app session intact,
        // and the next move asks again.
        match ops.begin_os_drag(data, None, dragging.kind) {
            crate::drag_payload::OsDragStart::Declined => false,
            crate::drag_payload::OsDragStart::Pending(drag) => {
                self.os_drag_pending = Some(drag);
                false
            }
            crate::drag_payload::OsDragStart::Started => {
                self.hand_session_to_os(None, &mut *ops);
                true
            }
        }
    }

    /// The platform has started the native drag `drag` this window asked for
    /// and answered [`OsDragStart::Pending`](crate::drag_payload::OsDragStart::Pending)
    /// about: hand the in-app session over to it. Routed here by `teksilo-app`
    /// when a backend reports the start (`ExternalDragEvent::DragStarted`, or,
    /// for the Windows modal drag loop, just before the loop runs).
    ///
    /// A confirmation for any other drag is stale: its in-app drag already
    /// ended (a release, Escape) and asked the platform to drop the start.
    /// When nothing else is waiting or running here, the platform is asked
    /// again to end a native drag that nothing here owns; otherwise the newer
    /// drag's start replaces it on the platform's side.
    pub fn confirm_os_drag(
        &mut self,
        drag: crate::drag_payload::OsDragId,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        if self.os_drag_pending == Some(drag) {
            self.os_drag_pending = None;
            if self.active_drag.is_none() {
                ops.cancel_os_drag();
                return;
            }
            self.hand_session_to_os(Some(drag), &mut *ops);
            return;
        }
        if self.os_drag_pending.is_none()
            && self.outbound_drag_source.is_none()
            && !self.os_drag_reentered
        {
            ops.cancel_os_drag();
        }
    }

    /// The platform has the pointer: take the in-app session, park its typed
    /// payload for the OS drag's lifetime and revoke what the press had going.
    fn hand_session_to_os(
        &mut self,
        drag: Option<crate::drag_payload::OsDragId>,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        let Some(dragging) = self.active_drag.as_ref().map(|d| d.pointer) else {
            return;
        };
        self.os_drag_pending = None;
        self.outbound_drag_id = drag;
        self.reentered_drop_outcome = None;
        // Escalated: the OS owns the drag now. Take the in-app session and park
        // its full (typed) payload in the app-global stash for the OS drag's
        // lifetime, so any window the drag re-enters can recover it. Remember
        // the source so the eventual `DragEnded` notifies it.
        let prev_target = self.active_drag.as_ref().and_then(|d| d.current_target);
        self.cleanup_drag_preview();
        let drag = self
            .active_drag
            .take()
            .expect("active_drag present (checked above)");
        self.os_drop_accepted = None;
        self.release_drag_capture(drag.source_widget);
        self.current_cursor = crate::widget::CursorIcon::Default;
        self.outbound_drag_source = drag.source_widget;
        outbound_begin(drag.payload, dragging);
        if let Some(prev) = prev_target {
            self.fire_on_drag_leave(prev, &mut *ops);
        }
        // The OS owns the drag from here. On Wayland, macOS and Windows this
        // window sees no further move and no `Up` for it, because the release
        // happens over whatever the drag was dropped on; on X11 winit keeps its
        // implicit grab and this window still gets them, and the router keeps
        // them away from a re-entered session (the XDND events drive it) and
        // swallows the `Up` of the press cancelled below. The source is told through
        // `on_drag_ended(OsCopy | OsMove | Cancelled)` when the OS reports back,
        // but anything *else* this press had going (a recognizer mid-drag, an
        // ancestor still competing) has to be revoked now.
        //
        // The pointer to revoke is the **session's**, not `current_pointer_id`:
        // the handover can be reached from a drag tick (a tick that scrolls can
        // move the reported position outside the window) or from a platform
        // confirmation, both outside any sample, where the singular accessor
        // names the mouse. Cancelling the mouse there would leave the real
        // contact armed.
        self.cancel_pointer_to(
            dragging.id,
            crate::pointer::CancelReason::OsDragStarted,
            self.outbound_drag_source,
            &mut *ops,
        );
    }

    /// A re-entered OS drag left this window again: return the typed payload to
    /// the app-global stash and tear down the internal session, *without* a
    /// terminal `on_drag_ended` (the OS drag is still in flight).
    fn reexit_outbound(&mut self, ops: &mut dyn crate::window::WindowOps) {
        let prev_target = self.active_drag.as_ref().and_then(|d| d.current_target);
        self.cleanup_drag_preview();
        let source = self.active_drag.as_ref().and_then(|d| d.source_widget);
        if let Some(drag) = self.active_drag.take() {
            outbound_restash(drag.payload);
        }
        self.os_drag_reentered = false;
        self.reentered_drop_outcome = None;
        self.os_drop_accepted = None;
        self.release_drag_capture(source);
        self.current_cursor = crate::widget::CursorIcon::Default;
        if let Some(prev) = prev_target {
            self.fire_on_drag_leave(prev, &mut *ops);
        }
    }

    /// Resolve an OS (outbound) drag at its terminal event, for a backend
    /// whose events name no drag (macOS, which starts at once). See
    /// [`Self::handle_os_drag_ended_for`].
    pub fn handle_os_drag_ended(
        &mut self,
        outcome: crate::drag_payload::DropOutcome,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        self.handle_os_drag_ended_for(None, outcome, ops);
    }

    /// Resolve an OS (outbound) drag at its terminal event. Clears the global
    /// typed-payload stash and fires `on_drag_ended(outcome)` once on the
    /// source widget (set only on the window that started the drag). Routed
    /// here by `teksilo-app` when the platform backend reports `DragEnded`.
    ///
    /// `drag` names the drag the event is about, for a backend that answered
    /// [`OsDragStart::Pending`](crate::drag_payload::OsDragStart::Pending). The
    /// end of a start this window is still waiting for (no press serial, no
    /// selection ownership) ends the in-app drag as the platform says. The
    /// end of a drag that is neither waiting nor running here is stale and
    /// changes nothing: it belongs to a drag released before its start,
    /// whose late event must not end the next one.
    pub fn handle_os_drag_ended_for(
        &mut self,
        drag: Option<crate::drag_payload::OsDragId>,
        outcome: crate::drag_payload::DropOutcome,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        let waiting = match drag {
            Some(id) if self.os_drag_pending == Some(id) => true,
            Some(id) if self.outbound_drag_id == Some(id) => false,
            Some(id) => {
                crate::trace_input!(Gestures, "a stale end of OS drag {} is ignored", id.raw());
                return;
            }
            None => self.os_drag_pending.is_some(),
        };
        // A start this window was waiting for ended without starting: the drag
        // ends as the platform says, and the in-app session with it, as it
        // would have if the start had been answered at once. Nothing of it was
        // parked, so the stash (perhaps an earlier drag's, still ending) stays.
        if waiting {
            self.os_drag_pending = None;
            self.reentered_drop_outcome = None;
            self.cancel_active_drag(&mut *ops);
            return;
        }
        // The OS guarantees this terminal event; end the stash so a later drag
        // from another app can't be mistaken for ours.
        outbound_end();
        self.os_drag_reentered = false;
        self.outbound_drag_id = None;
        // A drop back on this window was an in-app drop, whatever the platform
        // reports.
        let outcome = self.reentered_drop_outcome.take().unwrap_or(outcome);
        if let Some(source) = self.outbound_drag_source.take() {
            self.fire_on_drag_ended(source, outcome, &mut *ops);
        }
    }

    /// Abort any outbound OS drag this tree participates in, used when the
    /// window is closing. If this tree is the drag *source*, the whole drag is
    /// ending (the source object dies with the window) — end the stash so a
    /// later genuine external drag can't be mistaken for ours. If instead this
    /// is a non-source window currently holding the re-entered payload, hand it
    /// back to the stash so another window can still recover it. No
    /// `on_drag_ended` fires (the window and its handlers are being torn down).
    pub fn abort_outbound_drag(&mut self) {
        if self.outbound_drag_source.take().is_some() {
            outbound_end();
        } else if self.os_drag_reentered
            && let Some(drag) = self.active_drag.take()
        {
            outbound_restash(drag.payload);
        }
        self.os_drag_reentered = false;
        self.os_drag_pending = None;
        self.outbound_drag_id = None;
        self.reentered_drop_outcome = None;
        self.os_drop_accepted = None;
    }

    /// Update the drag session on pointer move: find the drop target under the
    /// pointer and call its `on_drag_hover` handler. For a move that is not a
    /// pointer sample (a drag tick, a scroll, an OS drag phase); a real sample
    /// goes through [`Self::handle_drag_sample`].
    pub(super) fn handle_drag_move(
        &mut self,
        position: teksilo_canvas::Point,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        self.drag_move(position, false, ops);
    }

    /// [`Self::handle_drag_move`] for a real pointer sample, the only move that
    /// may hand an [`OsHandoff::Eager`](crate::drag_payload::OsHandoff::Eager)
    /// drag to the platform while the pointer is inside the window.
    pub(super) fn handle_drag_sample(
        &mut self,
        position: teksilo_canvas::Point,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        self.drag_move(position, true, ops);
    }

    fn drag_move(
        &mut self,
        position: teksilo_canvas::Point,
        sample: bool,
        ops: &mut dyn crate::window::WindowOps,
    ) {
        // Update position on the session
        if let Some(ref mut drag) = self.active_drag {
            drag.current_position = position;
        }

        // If an internal, OS-exportable drag is due for the platform, hand it
        // over as a native drag and stop the in-app pipeline.
        if self.try_escalate_to_os_drag(position, sample, &mut *ops) {
            return;
        }

        // Update preview overlay placement. `update_placement` only
        // stores the new enum — the actual overlay bounds are recomputed
        // by `position_overlays` which runs inside `WidgetTree::layout()`
        // behind a `needs_layout` gate. Mark the content widget dirty so
        // the next layout pass actually re-positions the preview instead
        // of leaving it pinned at (0, 0).
        let preview_content = self
            .active_drag
            .as_ref()
            .and_then(|d| Some((d.preview_overlay_id?, d.preview_content_id?)));
        if let Some((overlay_id, content_id)) = preview_content {
            // Never *under* the contact for a coarse pointer: a preview pinned
            // to the point a finger reported is behind the hand that is
            // carrying it, so the user drags a card they cannot see. One branch,
            // in `OverlayPlacement::at_pointer_for`, shared with every
            // point-anchored panel — a mouse keeps `AtPointer` byte for byte.
            let dragging = self.active_drag.as_ref().map(|d| d.pointer);
            let placement = match dragging {
                Some(pointer) => {
                    crate::overlay::OverlayPlacement::at_pointer_for(position, &pointer)
                }
                None => crate::overlay::OverlayPlacement::AtPointer(position),
            };
            self.overlay_manager.update_placement(overlay_id, placement);
            self.arena.mark_needs_layout(content_id);
        }

        // Hit-test to find the widget under the pointer, excluding the drag
        // preview overlay and its content widget so they don't block hit-testing
        // of actual drop targets.
        let exclude_overlay = self.active_drag.as_ref().and_then(|d| d.preview_overlay_id);
        let exclude_widget = self.active_drag.as_ref().and_then(|d| d.preview_content_id);
        // Routed for the pointer dragging: a finger reaches a small drop target
        // through the same widening its press would have used, so hover and
        // drop agree with each other and with a plain tap.
        let dragging = self.current_input.pointer;
        let target =
            self.hit_test_for_excluding(position, &dragging, exclude_overlay, exclude_widget);

        // Drop-target bubbling: walk up from the hit target through successive
        // drop targets, firing each one's `on_drag_hover`, and stop at the first
        // that ENGAGES (returns a non-`NoFeedback` response). A target that
        // returns `NoFeedback` does not accept this payload, so the drag bubbles
        // to the next drop target above it — letting a reorderable view behind a
        // per-row `DropTarget` still receive the drag. Pointer position is passed
        // to each handler in TARGET-LOCAL coordinates.
        let mut candidate = target.and_then(|t| self.find_drop_target_at_or_above(t));
        let mut engaged: Option<WidgetId> = None;
        let mut engaged_feedback = crate::drag_state::DropFeedback::NoFeedback;
        let mut bubbled_past: Vec<WidgetId> = Vec::new();
        while let Some(cand) = candidate {
            let fb = self.fire_on_drag_hover(cand, position, &mut *ops);
            if fb.is_engaged() {
                engaged = Some(cand);
                engaged_feedback = fb;
                break;
            }
            bubbled_past.push(cand);
            candidate = self.next_drop_target_above(cand);
        }

        // Resolve the tracked target and clear stray hover state:
        // - If an ancestor ENGAGED, every rejecting target we passed is
        //   transparent (the drag is accepted above) — clear them all so none
        //   leaves a stuck "forbidden" border.
        // - If NOTHING engaged, the drag is genuinely rejected: the DEEPEST drop
        //   target keeps its own reject affordance and becomes the tracked target
        //   (cleared when the drag moves off); clear only the ancestors above it.
        let (new_target, new_feedback) = if engaged.is_some() {
            for cand in &bubbled_past {
                self.fire_on_drag_leave(*cand, &mut *ops);
            }
            (engaged, engaged_feedback)
        } else if let Some((&deepest, rest)) = bubbled_past.split_first() {
            for cand in rest {
                self.fire_on_drag_leave(*cand, &mut *ops);
            }
            (Some(deepest), crate::drag_state::DropFeedback::NoFeedback)
        } else {
            (None, crate::drag_state::DropFeedback::NoFeedback)
        };

        // Fire `on_drag_leave` on the previously-tracked target when it changes.
        // Skip targets already cleared by the per-frame rejecter cleanup above:
        // a target that rejected this frame while an ancestor engaged is in
        // `bubbled_past` and has already had its `on_drag_leave` fired, so
        // re-firing here would deliver two leaves for one pointer move.
        let prev_target = self.active_drag.as_ref().and_then(|d| d.current_target);
        if prev_target != new_target
            && let Some(prev) = prev_target
            && !bubbled_past.contains(&prev)
        {
            self.fire_on_drag_leave(prev, &mut *ops);
        }
        let engaged_now = new_feedback.is_engaged();
        // A re-entered app drag counts: the OS still owns it, its offer is still
        // negotiating, and a refusal there must still show the refusing cursor.
        // It is not `is_external` — the session was rebuilt as an internal one so
        // in-app targets see the typed payload — which is exactly why the flag
        // alone would have missed the app's own cross-window drag.
        let is_os_drag =
            self.os_drag_reentered || self.active_drag.as_ref().is_some_and(|d| d.is_external);
        if let Some(ref mut drag) = self.active_drag {
            drag.current_target = new_target;
            drag.feedback = new_feedback;
        }
        // Revise the OS's own accept state from the widget's verdict.
        //
        // An inbound backend has to answer the source *synchronously* — XDND
        // requires an `XdndStatus` per position and Wayland wants an
        // `accept` + `set_actions` on the offer — long before the widget tree
        // has seen the sample, so its first answer can only be about format
        // compatibility. That is why the cursor showed "will accept" over a
        // target that rejects: nothing ever told the OS otherwise. Pushed only
        // on a change, because the OS side is a round trip per call and a
        // motion stream would otherwise re-send the same answer every sample.
        if is_os_drag && self.os_drop_accepted != Some(engaged_now) {
            self.os_drop_accepted = Some(engaged_now);
            ops.set_drop_accepted(engaged_now);
        }
    }

    /// Fire `on_drag_hover` on a single drop target and return its response.
    /// A drop target that has an `on_drop` handler but no `on_drag_hover`
    /// engages optimistically (`Accept`, no visual) so it can still receive the
    /// drop; `on_drop` makes the final decision on release.
    fn fire_on_drag_hover(
        &mut self,
        target_id: WidgetId,
        position: teksilo_canvas::Point,
        ops: &mut dyn crate::window::WindowOps,
    ) -> crate::drag_state::DropFeedback {
        use crate::drag_state::DropFeedback;
        let target_bounds = self.arena.bounds(target_id);
        let local =
            teksilo_canvas::Point::new(position.x - target_bounds.x, position.y - target_bounds.y);
        let (mut ext_handler, mut own_handler, has_on_drop) = match self.arena.get_mut(target_id) {
            Some(node) => {
                let has_on_drop = node.any_handler(|h| h.on_drop.is_some());
                let ext = node.external_handlers.on_drag_hover.take();
                let own = node.handlers.on_drag_hover.take();
                (ext, own, has_on_drop)
            }
            None => return DropFeedback::NoFeedback,
        };
        // Drop-only target (no hover handler): engage optimistically.
        if ext_handler.is_none() && own_handler.is_none() {
            return if has_on_drop {
                DropFeedback::Accept
            } else {
                DropFeedback::NoFeedback
            };
        }
        let mut feedback = DropFeedback::NoFeedback;
        if self.active_drag.is_some() {
            let mut ctx = self.make_event_context(&mut *ops);
            if let Some(ref drag) = self.active_drag {
                if let Some(h) = ext_handler.as_mut() {
                    feedback = h(&drag.payload, local, &mut ctx);
                }
                if let Some(h) = own_handler.as_mut() {
                    feedback = h(&drag.payload, local, &mut ctx);
                }
            }
            if let Some(node) = self.arena.get_mut(target_id) {
                node.external_handlers.on_drag_hover = ext_handler;
                node.handlers.on_drag_hover = own_handler;
            }
            self.collect_from_ctx(ctx, target_id);
            self.arena.mark_needs_paint(target_id);
        } else if let Some(node) = self.arena.get_mut(target_id) {
            node.external_handlers.on_drag_hover = ext_handler;
            node.handlers.on_drag_hover = own_handler;
        }
        feedback
    }

    /// The next drop target strictly above `id` (its nearest ancestor with a
    /// drop handler) — used to bubble a drag past a non-accepting target.
    fn next_drop_target_above(&self, id: WidgetId) -> Option<WidgetId> {
        let parent = self.arena.parent(id)?;
        self.find_drop_target_at_or_above(parent)
    }

    /// Complete the drag: fire `on_drop` on the target widget and end the
    /// session. Returns how the drag ended, `None` when there was no session.
    pub(super) fn handle_drag_drop(
        &mut self,
        position: teksilo_canvas::Point,
        ops: &mut dyn crate::window::WindowOps,
    ) -> Option<crate::drag_payload::DropOutcome> {
        // Released while the platform was still starting the native drag the
        // in-app drag asked for: the drag ends as `Cancelled`, with no drop.
        // The drop belongs to whatever the native drag would have been dropped
        // on, which nothing knows yet, so landing it in-app could be wrong.
        if self.os_drag_pending.is_some() && self.active_drag.is_some() {
            self.cancel_active_drag(&mut *ops);
            return Some(crate::drag_payload::DropOutcome::Cancelled);
        }

        // Clean up preview overlay
        self.cleanup_drag_preview();

        self.active_drag.as_ref()?;

        // Determine the drop target while the session is still live. Normally
        // it's the target the last hover ENGAGED (drop-target bubbling already
        // chose it). For a drop with no prior hover (a quick drag, or a
        // programmatic `start_drag` + release), re-run the bubbling engagement at
        // the drop position so the drop still lands — and bubbles past a
        // non-accepting per-row target exactly as a hover would.
        // Ignore a `current_target` whose widget was destroyed since the last
        // hover (a rebuild tore it down mid-drag) — otherwise the drop resolves
        // to a dead arena id and is silently lost. Fall through to the
        // re-hit-test below so the drop still lands on whatever is live now.
        let mut drop_target = self
            .active_drag
            .as_ref()
            .and_then(|d| d.current_target)
            .filter(|&t| self.arena.is_active(t));
        if drop_target.is_none() {
            let dropping = self.current_input.pointer;
            let hit = self.hit_test_for(position, &dropping);
            let mut candidate = hit.and_then(|t| self.find_drop_target_at_or_above(t));
            while let Some(cand) = candidate {
                if self
                    .fire_on_drag_hover(cand, position, &mut *ops)
                    .is_engaged()
                {
                    drop_target = Some(cand);
                    break;
                }
                // Clear the bubbled-past target's hover state so it doesn't stay
                // highlighted after the drag ends.
                self.fire_on_drag_leave(cand, &mut *ops);
                candidate = self.next_drop_target_above(cand);
            }
        }

        // Take the drag session
        let drag = self.active_drag.take()?;
        self.os_drop_accepted = None;
        self.release_drag_capture(drag.source_widget);
        self.current_cursor = crate::widget::CursorIcon::Default;
        // Source widget so an in-app drop notifies its originator via
        // `on_drag_ended`. External drags carry no source.
        let source = drag.source_widget;
        // Default: landed on nothing ⇒ cancelled. Set to `InApp { accepted }`
        // when a drop handler actually runs.
        let mut outcome = crate::drag_payload::DropOutcome::Cancelled;

        // Fire on_drag_leave on the engaged target before on_drop runs — widgets
        // own their feedback state and must be given a chance to clear it
        // regardless of whether the drop is accepted.
        if let Some(prev) = drop_target {
            self.fire_on_drag_leave(prev, &mut *ops);
        }

        // on_drop is a "decision" handler (returns bool). Prefer own over
        // external: the widget's own drop semantics trump any external
        // listener. If the own bucket doesn't have it, fall back to
        // external. Fires exactly once, not both.
        if let Some(target_id) = drop_target {
            let target_bounds = self.arena.bounds(target_id);
            let local = teksilo_canvas::Point::new(
                position.x - target_bounds.x,
                position.y - target_bounds.y,
            );
            let (taken_own, taken_ext) = match self.arena.get_mut(target_id) {
                Some(node) => {
                    let own = node.handlers.on_drop.take();
                    let ext = if own.is_none() {
                        node.external_handlers.on_drop.take()
                    } else {
                        None
                    };
                    (own, ext)
                }
                None => (None, None),
            };
            let picked = if let Some(h) = taken_own {
                Some((h, /*is_own=*/ true))
            } else {
                taken_ext.map(|h| (h, /*is_own=*/ false))
            };
            if let Some((mut handler, is_own)) = picked {
                let mut ctx = self.make_event_context(&mut *ops);
                let accepted = handler(drag.payload, local, &mut ctx);
                outcome = crate::drag_payload::DropOutcome::InApp { accepted };
                if let Some(node) = self.arena.get_mut(target_id) {
                    if is_own {
                        node.handlers.on_drop = Some(handler);
                    } else {
                        node.external_handlers.on_drop = Some(handler);
                    }
                }
                self.collect_from_ctx(ctx, target_id);
                self.arena.mark_needs_paint(target_id);
            }
        }
        // Notify the source the drag it started has ended (in-app drops only;
        // external drags carry no source). Payload was moved into the handler
        // above, or dropped (Rust Drop) if unaccepted.
        if let Some(src) = source {
            self.fire_on_drag_ended(src, outcome, &mut *ops);
        }
        Some(outcome)
    }

    /// Walk up from a widget to find the nearest ancestor (or self) with a
    /// drop handler (`on_drop` or `on_drag_hover`) in either bucket.
    fn find_drop_target_at_or_above(&self, start: WidgetId) -> Option<WidgetId> {
        let mut current = Some(start);
        while let Some(id) = current {
            if let Some(node) = self.arena.get(id)
                && node.any_handler(|h| h.on_drop.is_some() || h.on_drag_hover.is_some())
            {
                return Some(id);
            }
            current = self.arena.parent(id);
        }
        None
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::test_widgets::{FillWidget, StackWidget};
    use crate::widget::CursorIcon;
    use crate::widget_builder::WidgetBuilder;

    #[test]
    fn start_drag_creates_session() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new().on_tap({
            move |_pos, ctx: &mut crate::widget::EventContext| {
                ctx.start_drag(
                    ctx.focus_requests.first().copied().unwrap_or_default(),
                    crate::drag_payload::DragPayload::typed(42_u32),
                );
            }
        }));
        tree.layout(SizeProposal::exact(100.0, 50.0));

        // Manually start a drag via EventContext
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);

        assert!(tree.active_drag.is_some());
        let drag = tree.active_drag.as_ref().unwrap();
        assert_eq!(drag.source_widget, Some(source));
        assert!(!drag.is_external);
        assert!(drag.payload.has_typed::<u32>());
    }

    #[test]
    fn drag_move_updates_position() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Start a drag session
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed("hello"));
        tree.collect_from_ctx(ctx, source);
        assert!(tree.active_drag.is_some());

        // Move the pointer
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(50.0, 30.0)));

        let drag = tree.active_drag.as_ref().unwrap();
        assert!((drag.current_position.x - 50.0).abs() < 0.01);
        assert!((drag.current_position.y - 30.0).abs() < 0.01);
    }

    #[test]
    fn escape_cancels_drag() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(99_i32));
        tree.collect_from_ctx(ctx, source);
        assert!(tree.active_drag.is_some());

        tree.press_key(Key::Escape, Modifiers::NONE);
        assert!(tree.active_drag.is_none(), "drag should be cancelled");
    }

    #[test]
    fn drop_on_target_fires_handler() {
        use std::cell::Cell;
        use std::rc::Rc;

        let dropped = Rc::new(Cell::new(false));
        let dropped_value = Rc::new(Cell::new(0_u32));
        let d = dropped.clone();
        let dv = dropped_value.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        // Target occupies right half (100..200, 0..100)
        let _target = tree.add(FillWidget::new().on_drop(move |mut payload, _pos, _ctx| {
            d.set(true);
            if let Some(val) = payload.take_typed::<u32>() {
                dv.set(val);
            }
            true
        }));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Start drag from source
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);

        // Drop at a position over the target
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(150.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(tree.active_drag.is_none(), "drag session should be cleared");
        assert!(dropped.get(), "on_drop should have been called");
        assert_eq!(dropped_value.get(), 42);
    }

    #[test]
    fn drop_on_no_target_cancels() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);

        // Drop outside any widget
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(999.0, 999.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(tree.active_drag.is_none(), "drag session should be cleared");
    }

    #[test]
    fn drop_falls_through_a_destroyed_current_target() {
        // A rebuild that destroys the hovered drop target mid-drag (e.g. a
        // docking side disabled while dragging over its rail) must not leave a
        // stale `current_target` that swallows the drop into a dead arena id.
        // The drop should re-hit-test and land on the live target beneath.
        use std::cell::Cell;
        use std::rc::Rc;

        let bg_dropped = Rc::new(Cell::new(false));
        let bg_sink = bg_dropped.clone();

        let mut tree = WidgetTree::new();
        // Children stack (topmost = last added). Source at the bottom (just the
        // drag origin), then the background drop target, then the foreground
        // drop target on top.
        let source = tree.add(FillWidget::new());
        let _bg = tree.add(FillWidget::new().on_drop(move |_p, _pos, _ctx| {
            bg_sink.set(true);
            true
        }));
        // Foreground drop target on top — engages on hover so it becomes the
        // drag's `current_target`.
        let fg = tree.add(
            FillWidget::new()
                .on_drag_hover(|_payload, _pos, _ctx| {
                    crate::drag_state::DropFeedback::InsertionLine {
                        y: 50.0,
                        width: 200.0,
                    }
                })
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(7_u32));
        tree.collect_from_ctx(ctx, source);

        // Hover over the foreground target → it becomes `current_target`.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        assert_eq!(
            tree.active_drag.as_ref().unwrap().current_target,
            Some(fg),
            "fg engaged as the current drop target"
        );

        // Tear the foreground target down mid-drag.
        tree.arena.destroy(fg);
        assert!(!tree.arena.is_active(fg));

        // Drop where fg used to be → must fall through to the live bg, not
        // vanish into the destroyed fg id.
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(100.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(tree.active_drag.is_none(), "drag session cleared");
        assert!(
            bg_dropped.get(),
            "drop landed on the live background target, not the destroyed one"
        );
    }

    /// Every competitor the current mouse press enrolled, innermost first.
    ///
    /// The successor to the deleted `armed_drag_observers()`: the same
    /// question, asked of the `PointerSequence` that replaced
    /// `drag_observers`.
    fn mouse_members(
        tree: &WidgetTree,
    ) -> Vec<(
        WidgetId,
        crate::gesture::MemberRole,
        crate::gesture::MemberState,
    )> {
        tree.sequence_members(crate::pointer::PointerId::MOUSE)
    }

    #[test]
    fn drag_arming_walks_to_an_ancestor_without_a_dead_zone() {
        // Baseline: pressing a button inside a draggable ancestor enrols the
        // ancestor as a `Gesture` member (so a press-drag can start the
        // ancestor drag — the cross-widget tap/drag disambiguation).
        use crate::gesture::{MemberRole, MemberState};
        let mut tree = WidgetTree::new();
        let button = tree.add(FillWidget::new().on_tap(|_e, _ctx| {}));
        let inner = tree.add(StackWidget::new().child(button));
        let ancestor = tree.add(StackWidget::new().child(inner).on_drag(|_phase, _ctx| {}));
        tree.layout(SizeProposal::exact(100.0, 100.0));

        let b = tree.bounds(button);
        tree.pointer_down_button(
            Point::new(b.x + b.width / 2.0, b.y + b.height / 2.0),
            PointerButton::Primary,
        );
        assert_eq!(
            mouse_members(&tree),
            vec![(ancestor, MemberRole::Gesture, MemberState::Possible)],
            "the draggable ancestor competes when the button press is not in a dead zone"
        );
        tree.pointer_up_button(
            Point::new(b.x + b.width / 2.0, b.y + b.height / 2.0),
            PointerButton::Primary,
        );
    }

    #[test]
    fn gesture_dead_zone_blocks_ancestor_drag_arming() {
        // The fix: a `gesture_dead_zone` boundary between the button and the
        // draggable ancestor stops the enrolment walk — the ancestor is NEVER
        // a member, so no amount of pointer jitter while clicking the button
        // can start the ancestor's drag (capture-release-proof, unlike a
        // recognizer-shadowing absorber).
        use crate::widget_builder::WidgetBuilder;
        let mut tree = WidgetTree::new();
        let button = tree.add(FillWidget::new().on_tap(|_e, _ctx| {}));
        let dead_zone = tree.add(StackWidget::new().child(button).gesture_dead_zone(true));
        let _ancestor = tree.add(
            StackWidget::new()
                .child(dead_zone)
                .on_drag(|_phase, _ctx| {}),
        );
        tree.layout(SizeProposal::exact(100.0, 100.0));

        let b = tree.bounds(button);
        tree.pointer_down_button(
            Point::new(b.x + b.width / 2.0, b.y + b.height / 2.0),
            PointerButton::Primary,
        );
        assert!(
            mouse_members(&tree).is_empty(),
            "a dead zone blocks the draggable ancestor from competing"
        );
        tree.pointer_up_button(
            Point::new(b.x + b.width / 2.0, b.y + b.height / 2.0),
            PointerButton::Primary,
        );
    }

    #[test]
    fn a_dead_zone_boundary_blocks_a_mouse_exactly_as_it_blocks_a_finger() {
        // `gesture_dead_zone` is NOT sugar for `touch_action(NONE)`: a mouse
        // ignores touch actions entirely, so the substitution would delete the
        // mouse behaviour the flag exists for. Same tree, same press, two
        // pointer kinds, one answer.
        use crate::pointer::{
            BackendDeviceKey, PointerIdAllocator, PointerInfo, PointerPhase, PointerSample,
        };
        use crate::widget_builder::WidgetBuilder;

        let mut tree = WidgetTree::new();
        let button = tree.add(FillWidget::new().on_tap(|_e, _ctx| {}));
        let dead_zone = tree.add(StackWidget::new().child(button).gesture_dead_zone(true));
        tree.add(
            StackWidget::new()
                .child(dead_zone)
                .on_drag(|_phase, _ctx| {}),
        );
        tree.layout(SizeProposal::exact(100.0, 100.0));
        let b = tree.bounds(button);
        let at = Point::new(b.x + b.width / 2.0, b.y + b.height / 2.0);

        tree.pointer_down_button(at, PointerButton::Primary);
        assert!(
            mouse_members(&tree).is_empty(),
            "the mouse enrols no ancestor across the dead zone"
        );
        tree.pointer_up_button(at, PointerButton::Primary);

        let contact = PointerIdAllocator::global().begin(BackendDeviceKey::DEFAULT, 41);
        let sample = |phase| PointerSample {
            pointer: PointerInfo::touch(contact, crate::pointer::EventTime::from_millis(1)),
            phase,
            position: at,
            button: None,
            modifiers: Modifiers::NONE,
            coalesced: Vec::new(),
        };
        tree.dispatch_pointer(sample(PointerPhase::Down));
        assert!(
            tree.sequence_members(contact).is_empty(),
            "and neither does a finger"
        );
        tree.dispatch_pointer(sample(PointerPhase::Up));
    }

    #[test]
    fn drag_hover_calls_on_drag_hover() {
        use std::cell::Cell;
        use std::rc::Rc;

        let hover_count = Rc::new(Cell::new(0));
        let hc = hover_count.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(move |_payload, _pos, _ctx| {
                    hc.set(hc.get() + 1);
                    crate::drag_state::DropFeedback::InsertionLine {
                        y: 50.0,
                        width: 200.0,
                    }
                })
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Start drag
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed("test"));
        tree.collect_from_ctx(ctx, source);

        // Move over the target
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(150.0, 50.0)));

        assert!(
            hover_count.get() > 0,
            "on_drag_hover should have been called"
        );
    }

    /// Regression: a rejecting per-row target nested under an engaging ancestor
    /// must receive exactly ONE `on_drag_leave` for the pointer move that flips
    /// the ancestor from idle to engaged — not two. The per-frame rejecter
    /// cleanup (it's in `bubbled_past`) and the tracked-target-change cleanup
    /// (it was last frame's `current_target`) used to fire independently.
    #[test]
    fn rejecter_under_engaging_ancestor_leaves_once() {
        use std::cell::Cell;
        use std::rc::Rc;

        let leaves = Rc::new(Cell::new(0));
        let lv = leaves.clone();
        // The ancestor only engages once we flip this between the two moves,
        // reproducing "frame 1 nothing engages, frame 2 the ancestor does".
        let engage = Rc::new(Cell::new(false));
        let eg = engage.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());

        // Deepest target: always rejects (NoFeedback), counts its leaves.
        let child = tree.add(
            FillWidget::new()
                .on_drag_hover(|_payload, _pos, _ctx| crate::drag_state::DropFeedback::NoFeedback)
                .on_drag_leave(move |_ctx| lv.set(lv.get() + 1)),
        );
        // Ancestor container wrapping the child: engages conditionally.
        let _ancestor = tree.add(StackWidget::new().child(child).on_drag_hover(
            move |_payload, _pos, _ctx| {
                if eg.get() {
                    crate::drag_state::DropFeedback::Accept
                } else {
                    crate::drag_state::DropFeedback::NoFeedback
                }
            },
        ));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(7_u32));
        tree.collect_from_ctx(ctx, source);

        // Frame 1: nothing engages → child becomes the tracked (rejecting) target.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        assert_eq!(
            tree.active_drag.as_ref().unwrap().current_target,
            Some(child)
        );
        assert_eq!(leaves.get(), 0, "no leave yet — child is freshly tracked");

        // Frame 2: ancestor engages while child still rejects.
        engage.set(true);
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(101.0, 50.0)));

        assert_eq!(
            leaves.get(),
            1,
            "child must receive exactly one on_drag_leave, not two"
        );
    }

    #[test]
    fn drop_outside_window_cancels() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);
        assert!(tree.active_drag.is_some());

        // PointerUp far outside any widget
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(-100.0, -100.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(tree.active_drag.is_none(), "drag should be cleared");
    }

    #[test]
    fn drop_target_rejects_wrong_type() {
        use std::cell::Cell;
        use std::rc::Rc;

        let accepted = Rc::new(Cell::new(false));
        let a = accepted.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        // Target only accepts String payloads
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(|payload, _pos, _ctx| {
                    if payload.has_typed::<String>() {
                        crate::drag_state::DropFeedback::InsertionLine {
                            y: 0.0,
                            width: 100.0,
                        }
                    } else {
                        crate::drag_state::DropFeedback::NoFeedback
                    }
                })
                .on_drop(move |payload, _pos, _ctx| {
                    if payload.has_typed::<String>() {
                        a.set(true);
                        true
                    } else {
                        false
                    }
                }),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Drag a u32 (not String)
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);

        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(150.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(!accepted.get(), "on_drop should reject wrong payload type");
    }

    #[test]
    fn inter_widget_drop_transfers_payload() {
        use std::cell::Cell;
        use std::rc::Rc;

        let received_value = Rc::new(Cell::new(0_u32));
        let rv = received_value.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(|_payload, _pos, _ctx| {
                    crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 100.0,
                    }
                })
                .on_drop(move |mut payload, _pos, _ctx| {
                    if let Some(val) = payload.take_typed::<u32>() {
                        rv.set(val);
                        true
                    } else {
                        false
                    }
                }),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Start drag from source with typed payload
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(777_u32));
        tree.collect_from_ctx(ctx, source);

        // Drop on target
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(150.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert_eq!(
            received_value.get(),
            777,
            "Target should receive the typed payload from source"
        );
    }

    #[test]
    fn drop_on_child_walks_up_to_ancestor_drop_target() {
        use crate::test_widgets::StackWidget;
        use std::cell::Cell;
        use std::rc::Rc;

        // Parent container with `on_drop`; child has no drop handler. The
        // framework should walk up from the hit target to find the parent.
        let parent_fired = Rc::new(Cell::new(false));
        let pf = parent_fired.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let child = tree.add(FillWidget::new());
        let _parent = tree.add(StackWidget::new().child(child).on_drop(
            move |_payload, _pos, _ctx| {
                pf.set(true);
                true
            },
        ));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Start a drag.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(1_u8));
        tree.collect_from_ctx(ctx, source);

        // Drop at the child's center. Hit test lands on the child; drop
        // should bubble up to the parent StackWidget.
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(100.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(
            parent_fired.get(),
            "Parent's on_drop should fire via ancestor walk"
        );
    }

    #[test]
    fn drop_bubbles_past_a_rejecting_child_to_ancestor() {
        use crate::test_widgets::StackWidget;
        use std::cell::Cell;
        use std::rc::Rc;

        // A child drop target that REJECTS this payload (its `on_drag_hover`
        // returns `NoFeedback` and `on_drop` returns `false`) must NOT swallow
        // the drag — it bubbles to the accepting parent. This is the
        // per-row-`DropTarget`-over-a-reorderable-view case.
        let child_drop = Rc::new(Cell::new(false));
        let parent_drop = Rc::new(Cell::new(false));
        let cd = child_drop.clone();
        let pd = parent_drop.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let child = tree.add(
            FillWidget::new()
                .on_drag_hover(|_p, _pos, _ctx| crate::drag_state::DropFeedback::NoFeedback)
                .on_drop(move |_p, _pos, _ctx| {
                    cd.set(true);
                    false // reject → the framework should bubble past
                }),
        );
        let _parent = tree.add(
            StackWidget::new()
                .child(child)
                .on_drop(move |_p, _pos, _ctx| {
                    pd.set(true);
                    true
                }),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(7_u8));
        tree.collect_from_ctx(ctx, source);

        // Hover over the child (its on_drag_hover runs → NoFeedback → bubble),
        // then release there.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(100.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(parent_drop.get(), "drop bubbles to the accepting ancestor");
        assert!(
            !child_drop.get(),
            "the rejecting child must not receive the drop"
        );
    }

    #[test]
    fn drag_preview_overlay_created_and_dismissed() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        let overlay_count_before = tree.overlay_manager().len();

        // Start drag with a preview widget.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);

        assert!(tree.active_drag.is_some(), "drag session should be active");
        assert!(
            tree.active_drag
                .as_ref()
                .unwrap()
                .preview_overlay_id
                .is_some(),
            "preview overlay id should be recorded"
        );
        assert_eq!(
            tree.overlay_manager().len(),
            overlay_count_before + 1,
            "overlay count should increase by one for the preview"
        );

        // Drop outside any target — cleanup should remove the overlay.
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(999.0, 999.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(tree.active_drag.is_none(), "drag session should be cleared");
        assert_eq!(
            tree.overlay_manager().len(),
            overlay_count_before,
            "preview overlay should be dismissed on drop"
        );
    }

    #[test]
    fn drag_preview_teardown_releases_what_its_content_registered() {
        #[derive(Debug)]
        struct SpinningPreview;
        impl Widget for SpinningPreview {
            fn build(&mut self, ctx: &mut crate::build_context::BuildContext) -> Vec<WidgetId> {
                let _ = ctx.animated_quad(crate::animated_quad::AnimatedQuadKind::SpinnerArc {
                    period: std::time::Duration::from_secs(1),
                    arc_fraction: 0.25,
                    stroke_fraction: 0.12,
                    color: teksilo_tokens::Color::RED.into(),
                });
                Vec::new()
            }
            fn layout_response(
                &self,
                _: SizeProposal,
                _: &crate::widget::LayoutContext,
            ) -> crate::widget::LayoutResponse {
                teksilo_canvas::Size::new(20.0, 20.0).into()
            }
        }

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(SpinningPreview),
        );
        tree.collect_from_ctx(ctx, source);
        assert_eq!(tree.animated_quad_count(), 1, "the preview's build ran");

        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(999.0, 999.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));
        assert!(tree.active_drag.is_none());
        assert_eq!(
            tree.animated_quad_count(),
            0,
            "tearing the preview down must release its animated-quad slot"
        );
    }

    #[test]
    fn drag_preview_follows_pointer_position() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed("p"),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);

        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(73.0, 41.0)));

        let drag = tree.active_drag.as_ref().expect("active drag");
        assert!(
            (drag.current_position.x - 73.0).abs() < 0.01
                && (drag.current_position.y - 41.0).abs() < 0.01,
            "drag session position should track the pointer"
        );

        let overlay_id = drag.preview_overlay_id.expect("preview overlay");
        let overlay = tree
            .overlay_manager()
            .overlay(overlay_id)
            .expect("overlay looked up by id");
        match &overlay.placement {
            crate::overlay::OverlayPlacement::AtPointer(p) => {
                assert!(
                    (p.x - 73.0).abs() < 0.01 && (p.y - 41.0).abs() < 0.01,
                    "preview overlay placement should follow pointer"
                );
            }
            other => panic!("expected AtPointer placement, got {:?}", other),
        }
    }

    #[test]
    fn escape_during_hover_dismisses_preview() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(|_payload, _pos, _ctx| {
                    crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 100.0,
                    }
                })
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let overlay_count_before = tree.overlay_manager().len();

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);

        // Move over the target to establish feedback.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(150.0, 50.0)));

        assert!(tree.active_drag.is_some());
        assert_eq!(tree.overlay_manager().len(), overlay_count_before + 1);

        // Escape cancels: session cleared AND preview overlay dismissed.
        tree.press_key(Key::Escape, Modifiers::NONE);

        assert!(tree.active_drag.is_none(), "drag must be cancelled");
        assert_eq!(
            tree.overlay_manager().len(),
            overlay_count_before,
            "preview overlay must be dismissed after Escape"
        );
    }

    #[test]
    fn active_drag_blocks_on_tap_on_other_widgets() {
        use std::cell::Cell;
        use std::rc::Rc;

        // While a drag is in progress, PointerMove and PointerUp must go
        // through the drag pipeline (handle_drag_move / handle_drag_drop) —
        // NOT be dispatched to the hovered widget. A widget with `on_tap` in
        // the drop location should not receive it.
        let tap_fired = Rc::new(Cell::new(false));
        let tf = tap_fired.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _other = tree.add(FillWidget::new().on_tap(move |_pos, _ctx| {
            tf.set(true);
        }));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);

        // Move over and release on the `on_tap` widget. Normally this would
        // synthesize a Tap gesture — but an active drag short-circuits.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(150.0, 50.0)));
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(150.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert!(
            !tap_fired.get(),
            "on_tap must not fire during an active drag"
        );
    }

    // --- on_drag_leave lifecycle ---------------------------------------

    #[test]
    fn on_drag_leave_fires_when_pointer_leaves_target_bounds() {
        // Single drop target wrapped in an InsetWidget so its bounds do
        // NOT fill the viewport — the pointer can be "inside the scene
        // but outside the target" so a target-change (target → None) is
        // reachable without destroying widgets. That is the main
        // semantic we want `on_drag_leave` to cover.
        use crate::test_widgets::InsetWidget;
        use std::cell::Cell;
        use std::rc::Rc;

        let leave = Rc::new(Cell::new(0_u32));
        let l = leave.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        let _wrapper = tree.add(InsetWidget::new(40.0).set_child(target));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);

        // Pointer inside the inset (where the target lives).
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        assert_eq!(leave.get(), 0, "no leave yet — target just became active");

        // Pointer in the inset area, outside the target's bounds — the
        // only hit is the InsetWidget which has no drag handlers, so
        // drop_target becomes None. Target changed → leave fires on the
        // old target.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(10.0, 10.0)));
        assert_eq!(
            leave.get(),
            1,
            "on_drag_leave fires when pointer exits the target's bounds"
        );

        // Moving back in shouldn't fire again.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        assert_eq!(
            leave.get(),
            1,
            "leave fires at most once per leave transition"
        );

        // Leaving again fires a second time.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(10.0, 10.0)));
        assert_eq!(leave.get(), 2);
    }

    #[test]
    fn on_drag_leave_fires_on_drop() {
        use std::cell::Cell;
        use std::rc::Rc;

        let leave = Rc::new(Cell::new(0_u32));
        let l = leave.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(100.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert_eq!(leave.get(), 1, "on_drag_leave fires exactly once on drop");
    }

    #[test]
    fn on_drag_leave_fires_on_escape_cancel() {
        use std::cell::Cell;
        use std::rc::Rc;

        let leave = Rc::new(Cell::new(0_u32));
        let l = leave.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        tree.press_key(Key::Escape, Modifiers::NONE);

        assert_eq!(
            leave.get(),
            1,
            "Escape cancel must fire on_drag_leave on the current target"
        );
    }

    #[test]
    fn on_drag_leave_fires_when_source_destroyed_mid_drag() {
        use std::cell::Cell;
        use std::rc::Rc;

        let leave = Rc::new(Cell::new(0_u32));
        let l = leave.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));

        tree.arena.destroy(source);
        // revalidate_interaction_state runs on the next process_pending_rebuilds
        // — drive it by a no-op layout call.
        tree.layout(SizeProposal::exact(200.0, 100.0));

        assert!(
            tree.active_drag.is_none(),
            "active drag should have been cancelled"
        );
        assert_eq!(
            leave.get(),
            1,
            "on_drag_leave fires on the drop target when the source is torn down"
        );
    }

    #[test]
    fn on_drag_tick_fires_per_layout_pass() {
        use std::cell::Cell;
        use std::rc::Rc;

        let ticks = Rc::new(Cell::new(0_u32));
        let t = ticks.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_drag_tick(move |_pos, _ctx| t.set(t.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);
        // Move over the target so it becomes the current drop target.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));
        assert_eq!(ticks.get(), 0, "tick shouldn't have fired yet");

        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert_eq!(ticks.get(), 1);
        tree.layout(SizeProposal::exact(200.0, 100.0));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert_eq!(ticks.get(), 3);

        // End the drag; ticks stop.
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(100.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));
        let after_drop = ticks.get();
        tree.layout(SizeProposal::exact(200.0, 100.0));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert_eq!(
            ticks.get(),
            after_drop,
            "on_drag_tick must not fire after drag ends"
        );
    }

    #[test]
    fn on_drag_hover_and_on_drop_receive_widget_local_coordinates() {
        // Regression for "drop indicator is always 2 items below the
        // cursor": `on_drag_hover` and `on_drop` must receive the
        // pointer in the target's local coordinates, not tree coords.
        // Otherwise a widget placed below a header computes insertion
        // indices against an absolute Y and the line renders offset by
        // the header's height divided by row height.
        use crate::test_widgets::InsetWidget;
        use std::cell::Cell;
        use std::rc::Rc;

        let hover_local = Rc::new(Cell::new(Point::new(-1.0, -1.0)));
        let drop_local = Rc::new(Cell::new(Point::new(-1.0, -1.0)));
        let h = hover_local.clone();
        let d = drop_local.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        // Inset 40 pushes the drop target to (40, 40) in tree coords.
        let target = tree.add(
            FillWidget::new()
                .on_drag_hover(move |_p, pos, _ctx| {
                    h.set(pos);
                    crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    }
                })
                .on_drop(move |_payload, pos, _ctx| {
                    d.set(pos);
                    true
                }),
        );
        let _wrapper = tree.add(InsetWidget::new(40.0).set_child(target));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);

        // Move pointer to (100, 60) in tree coords — inside the inset
        // target whose origin is (40, 40). Local position should be
        // (60, 20).
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 60.0)));
        let hov = hover_local.get();
        assert!(
            (hov.x - 60.0).abs() < 0.01 && (hov.y - 20.0).abs() < 0.01,
            "on_drag_hover should receive local coords, got {:?}",
            hov,
        );

        // Drop at (110, 55) tree coords → local (70, 15).
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(110.0, 55.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));
        let drp = drop_local.get();
        assert!(
            (drp.x - 70.0).abs() < 0.01 && (drp.y - 15.0).abs() < 0.01,
            "on_drop should receive local coords, got {:?}",
            drp,
        );
    }

    #[test]
    fn active_drag_sets_grabbing_cursor() {
        // Starting a drag with a preview must switch the tree's cursor
        // to `Grabbing`; dropping or cancelling must reset to `Default`.
        // teksilo-app applies the tree's cursor to the winit window after
        // each pointer event, so this is what the user actually sees.
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        assert_eq!(tree.current_cursor(), CursorIcon::Default);

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);
        assert_eq!(tree.current_cursor(), CursorIcon::Grabbing);

        // Drop somewhere.
        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(50.0, 25.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));
        assert_eq!(tree.current_cursor(), CursorIcon::Default);
    }

    #[test]
    fn escape_cancel_resets_cursor() {
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);
        assert_eq!(tree.current_cursor(), CursorIcon::Grabbing);

        tree.press_key(Key::Escape, Modifiers::NONE);
        assert_eq!(tree.current_cursor(), CursorIcon::Default);
    }

    #[test]
    fn drag_preview_composite_gets_built() {
        // Regression — composite preview widgets must have their `build()`
        // called after `start_drag_with_preview`. A plain `arena.insert`
        // inserts the node but never runs build, leaving the preview tree
        // empty (no children, zero area of useful content) and the overlay
        // invisible. The fix routes through `add_boxed` so build fires.
        use std::cell::Cell;
        use std::rc::Rc;

        let built = Rc::new(Cell::new(false));
        let b = built.clone();

        #[derive(Debug)]
        struct CheckingWidget {
            built: Rc<Cell<bool>>,
        }
        impl Widget for CheckingWidget {
            fn build(&mut self, _ctx: &mut crate::build_context::BuildContext) -> Vec<WidgetId> {
                self.built.set(true);
                Vec::new()
            }
            fn layout_response(
                &self,
                _: SizeProposal,
                _: &crate::widget::LayoutContext,
            ) -> crate::widget::LayoutResponse {
                teksilo_canvas::Size::new(50.0, 20.0).into()
            }
        }

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(CheckingWidget { built: b }),
        );
        tree.collect_from_ctx(ctx, source);

        assert!(built.get(), "preview's build() must fire on drag start");
    }

    #[test]
    fn preview_placement_drives_layout_needs() {
        // Regression for "preview stays at (0, 0)": each pointer move
        // during drag updates the overlay placement via
        // `update_placement`, but the overlay's bounds are only
        // recomputed by `position_overlays` inside `layout()` — which
        // early-returns when nothing is `needs_layout`. Verify the
        // drag path marks the preview content dirty so layout actually
        // runs.
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 200.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(0_u32),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);

        // Right after drag start, the preview content should need layout
        // so the first layout pass positions it.
        assert!(
            tree.needs_layout(),
            "drag start must mark preview content for layout"
        );
        tree.layout(SizeProposal::exact(200.0, 200.0));
        assert!(
            !tree.needs_layout(),
            "layout should have cleared dirty flag"
        );

        // A subsequent PointerMove must remark the preview so its
        // overlay bounds get repositioned on the next layout pass.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(75.0, 120.0)));
        assert!(
            tree.needs_layout(),
            "PointerMove during drag must mark preview for layout"
        );
    }

    #[test]
    fn scroll_during_drag_routes_to_drop_target() {
        use std::cell::Cell;
        use std::rc::Rc;

        let scroll_count = Rc::new(Cell::new(0_u32));
        let sc = scroll_count.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::InsertionLine {
                        y: 0.0,
                        width: 10.0,
                    },
                )
                .on_scroll(move |event, _ctx| match event {
                    WidgetEvent::Scroll { .. } => {
                        sc.set(sc.get() + 1);
                        EventResponse::Handled
                    }
                    _ => EventResponse::Ignored,
                })
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);
        // Make target the current drop target.
        tree.dispatch_event(WidgetEvent::pointer_move(Point::new(100.0, 50.0)));

        // A wheel event during drag should reach the drop target (not the
        // stale hover from before the drag started).
        tree.dispatch_event(WidgetEvent::scroll(
            crate::event::ScrollDelta::Pixels { x: 0.0, y: 40.0 },
            Default::default(),
        ));
        assert_eq!(
            scroll_count.get(),
            1,
            "Scroll during drag must route to the current drop target"
        );
    }

    // --- External (OS) drag-and-drop -----------------------------------

    #[test]
    fn external_drop_delivers_files_and_marks_external() {
        use crate::drag_payload::ExternalDropData;
        use std::cell::RefCell;
        use std::path::PathBuf;
        use std::rc::Rc;

        let dropped_files: Rc<RefCell<Vec<PathBuf>>> = Rc::new(RefCell::new(Vec::new()));
        let was_external = Rc::new(std::cell::Cell::new(false));
        let df = dropped_files.clone();
        let we = was_external.clone();

        let mut tree = WidgetTree::new();
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(|payload, _pos, _ctx| {
                    // External file drags are accepted with a highlight.
                    if payload.is_external() && !payload.files().is_empty() {
                        crate::drag_state::DropFeedback::HighlightRect {
                            rect: teksilo_canvas::Rect::new(0.0, 0.0, 10.0, 10.0),
                            color: teksilo_tokens::Color::WHITE,
                        }
                    } else {
                        crate::drag_state::DropFeedback::NoFeedback
                    }
                })
                .on_drop(move |payload, _pos, _ctx| {
                    we.set(payload.is_external());
                    *df.borrow_mut() = payload.files().to_vec();
                    true
                }),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut noop = crate::window::NoopWindowOps;
        let data = ExternalDropData {
            files: vec![PathBuf::from("/tmp/a.png"), PathBuf::from("/tmp/b.png")],
            ..Default::default()
        };
        tree.begin_external_drag(Point::new(100.0, 50.0), data, &mut noop);
        assert!(tree.active_drag.is_some());
        assert!(tree.active_drag.as_ref().unwrap().is_external);

        tree.update_external_drag(Point::new(110.0, 55.0), &mut noop);
        // Pass the same files again at drop — exercises the payload-refresh path.
        let drop_data = ExternalDropData {
            files: vec![PathBuf::from("/tmp/a.png"), PathBuf::from("/tmp/b.png")],
            ..Default::default()
        };
        tree.end_external_drag(Point::new(110.0, 55.0), drop_data, &mut noop);

        assert!(
            tree.active_drag.is_none(),
            "external drag must clear on drop"
        );
        assert!(was_external.get(), "payload should report external origin");
        assert_eq!(
            *dropped_files.borrow(),
            vec![PathBuf::from("/tmp/a.png"), PathBuf::from("/tmp/b.png")],
        );
    }

    #[test]
    fn external_drop_passes_local_coordinates() {
        use crate::drag_payload::ExternalDropData;
        use crate::test_widgets::InsetWidget;
        use std::cell::Cell;
        use std::path::PathBuf;
        use std::rc::Rc;

        let drop_local = Rc::new(Cell::new(Point::new(-1.0, -1.0)));
        let d = drop_local.clone();

        let mut tree = WidgetTree::new();
        // Inset 40 → target origin at (40, 40).
        let target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::HighlightRect {
                        rect: teksilo_canvas::Rect::new(0.0, 0.0, 10.0, 10.0),
                        color: teksilo_tokens::Color::WHITE,
                    },
                )
                .on_drop(move |_payload, pos, _ctx| {
                    d.set(pos);
                    true
                }),
        );
        let _wrapper = tree.add(InsetWidget::new(40.0).set_child(target));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut noop = crate::window::NoopWindowOps;
        let data = ExternalDropData {
            files: vec![PathBuf::from("/tmp/x")],
            ..Default::default()
        };
        // Drop at tree (110, 55) → target-local (70, 15).
        tree.begin_external_drag(Point::new(110.0, 55.0), data, &mut noop);
        tree.end_external_drag(
            Point::new(110.0, 55.0),
            ExternalDropData::default(),
            &mut noop,
        );

        let drp = drop_local.get();
        assert!(
            (drp.x - 70.0).abs() < 0.01 && (drp.y - 15.0).abs() < 0.01,
            "external on_drop should receive local coords, got {:?}",
            drp,
        );
    }

    #[test]
    fn cancel_external_drag_clears_session_and_fires_leave() {
        use crate::drag_payload::ExternalDropData;
        use std::cell::Cell;
        use std::path::PathBuf;
        use std::rc::Rc;

        let left = Rc::new(Cell::new(0_u32));
        let l = left.clone();

        let mut tree = WidgetTree::new();
        let _target = tree.add(
            FillWidget::new()
                .on_drag_hover(
                    |_p, _pos, _ctx| crate::drag_state::DropFeedback::HighlightRect {
                        rect: teksilo_canvas::Rect::new(0.0, 0.0, 10.0, 10.0),
                        color: teksilo_tokens::Color::WHITE,
                    },
                )
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut noop = crate::window::NoopWindowOps;
        let data = ExternalDropData {
            files: vec![PathBuf::from("/tmp/x")],
            ..Default::default()
        };
        tree.begin_external_drag(Point::new(100.0, 50.0), data, &mut noop);
        assert!(tree.active_drag.is_some());

        tree.cancel_external_drag(&mut noop);
        assert!(tree.active_drag.is_none(), "cancel must clear the session");
        assert_eq!(
            left.get(),
            1,
            "cancel must fire on_drag_leave on the target"
        );
    }

    #[test]
    fn external_drag_helpers_noop_without_session() {
        // update/end/cancel are no-ops when no external session is active.
        let mut tree = WidgetTree::new();
        let _t = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(100.0, 50.0));

        let mut noop = crate::window::NoopWindowOps;
        tree.update_external_drag(Point::new(10.0, 10.0), &mut noop);
        tree.end_external_drag(
            Point::new(10.0, 10.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut noop,
        );
        tree.cancel_external_drag(&mut noop);
        assert!(tree.active_drag.is_none());
    }

    // --- Outbound (app → OS) escalation + unified on_drag_ended ----------

    /// `WindowOps` sink that records `begin_os_drag` calls and reports a
    /// configurable success, standing in for the platform backend.
    struct RecordingWindowOps {
        started: std::rc::Rc<std::cell::RefCell<Vec<crate::drag_payload::OutboundDragData>>>,
        /// The pointer kind each `begin_os_drag` was told about, in order.
        started_kinds: std::rc::Rc<std::cell::RefCell<Vec<teksilo_tokens::PointerKind>>>,
        /// What every `begin_os_drag` answers. A `Pending` answer carries a
        /// fresh id each time, recorded in `pending_ids`.
        start: crate::drag_payload::OsDragStart,
        pending_ids: std::rc::Rc<std::cell::RefCell<Vec<crate::drag_payload::OsDragId>>>,
        cancels: std::rc::Rc<std::cell::Cell<usize>>,
        /// Every `set_drop_accepted` the tree pushed, in order.
        accepts: std::rc::Rc<std::cell::RefCell<Vec<bool>>>,
    }

    impl RecordingWindowOps {
        fn new(succeed: bool) -> Self {
            Self::answering(if succeed {
                crate::drag_payload::OsDragStart::Started
            } else {
                crate::drag_payload::OsDragStart::Declined
            })
        }

        fn answering(start: crate::drag_payload::OsDragStart) -> Self {
            Self {
                started: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
                started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
                start,
                pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
                cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
                accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            }
        }
    }
    impl crate::window::WindowOps for RecordingWindowOps {
        fn open_window(
            &mut self,
            _c: crate::window::WindowConfig,
        ) -> crate::window::TeksiloWindowId {
            panic!("not used in these tests")
        }
        fn find_window(&self, _s: &str) -> Option<crate::window::TeksiloWindowId> {
            None
        }
        fn window_state(
            &self,
            _id: crate::window::TeksiloWindowId,
        ) -> Option<crate::window::WindowState> {
            None
        }
        fn windows(&self) -> Vec<crate::window::WindowState> {
            Vec::new()
        }
        fn focus_window(&mut self, _id: crate::window::TeksiloWindowId) {}
        fn close_window_by_id(&mut self, _id: crate::window::TeksiloWindowId) {}
        fn begin_os_drag(
            &mut self,
            data: crate::drag_payload::OutboundDragData,
            _image: Option<crate::drag_payload::DragImageData>,
            pointer: teksilo_tokens::PointerKind,
        ) -> crate::drag_payload::OsDragStart {
            self.started.borrow_mut().push(data);
            self.started_kinds.borrow_mut().push(pointer);
            match self.start {
                crate::drag_payload::OsDragStart::Pending(_) => {
                    let id = crate::drag_payload::OsDragId::next();
                    self.pending_ids.borrow_mut().push(id);
                    crate::drag_payload::OsDragStart::Pending(id)
                }
                other => other,
            }
        }
        fn cancel_os_drag(&mut self) {
            self.cancels.set(self.cancels.get() + 1);
        }
        fn set_drop_accepted(&mut self, accepted: bool) {
            self.accepts.borrow_mut().push(accepted);
        }
    }

    fn exportable_payload() -> crate::drag_payload::DragPayload {
        crate::drag_payload::DragPayload::typed(7_u32).with_mime("text/plain", b"hi".to_vec())
    }

    #[test]
    fn internal_exportable_drag_escalates_when_leaving_window() {
        let started = std::rc::Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started: started.clone(),
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, exportable_payload());
        tree.collect_from_ctx(ctx, source);
        assert!(tree.active_drag.is_some());

        // Inside the window: no escalation.
        tree.handle_drag_move(Point::new(100.0, 50.0), &mut ops);
        assert!(started.borrow().is_empty());
        assert!(tree.active_drag.is_some());

        // Pointer leaves the window: escalate.
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1, "begin_os_drag called once");
        assert!(
            started.borrow()[0].mime.contains_key("text/plain"),
            "outbound data carries the payload's mime"
        );
        assert!(tree.active_drag.is_none(), "in-app session torn down");
        assert_eq!(tree.outbound_drag_source, Some(source));
    }

    #[test]
    fn os_drag_ended_fires_source_on_drag_ended() {
        use crate::drag_payload::DropOutcome;
        use std::cell::Cell;
        use std::rc::Rc;

        let outcome = Rc::new(Cell::new(None));
        let o = outcome.clone();

        let started = Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started,
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source =
            tree.add(FillWidget::new().on_drag_ended(move |outcome, _ctx| o.set(Some(outcome))));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, exportable_payload());
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops); // escalate
        assert_eq!(tree.outbound_drag_source, Some(source));

        tree.handle_os_drag_ended(DropOutcome::OsMove, &mut ops);
        assert_eq!(outcome.get(), Some(DropOutcome::OsMove));
        assert!(
            tree.outbound_drag_source.is_none(),
            "cleared after delivery"
        );
    }

    #[test]
    fn escape_during_an_escalated_drag_asks_the_platform_to_cancel() {
        // The in-app session is gone once the platform accepts the hand-off,
        // so this cannot ride the `active_drag` Escape path. Routing it through
        // `WindowOps` (rather than special-casing it in a backend's own event
        // loop) is what makes it observable here at all.
        let mut ops = RecordingWindowOps::new(true);
        let cancels = ops.cancels.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, exportable_payload());
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops); // escalate
        assert_eq!(tree.outbound_drag_source, Some(source));

        tree.dispatch_event_with_ops(
            crate::event::WidgetEvent::KeyDown {
                key: crate::event::Key::Escape,
                modifiers: crate::event::Modifiers::NONE,
                text: None,
            },
            &mut ops,
        );
        assert_eq!(cancels.get(), 1, "the platform must be asked to cancel");
        assert_eq!(
            tree.outbound_drag_source,
            Some(source),
            "the session stays until the backend reports its terminal outcome — \
             tearing it down here would drop the source's on_drag_ended"
        );
    }

    #[test]
    fn escape_without_an_os_drag_does_not_touch_the_platform() {
        let mut ops = RecordingWindowOps::new(true);
        let cancels = ops.cancels.clone();

        let mut tree = WidgetTree::new();
        tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        tree.dispatch_event_with_ops(
            crate::event::WidgetEvent::KeyDown {
                key: crate::event::Key::Escape,
                modifiers: crate::event::Modifiers::NONE,
                text: None,
            },
            &mut ops,
        );
        assert_eq!(cancels.get(), 0);
    }

    #[test]
    fn no_backend_keeps_session_active_on_leave() {
        // begin_os_drag returns false (no outbound backend): the in-app
        // drag stays active so the user can drag back in — current behavior.
        let started = std::rc::Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started: started.clone(),
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Declined,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, exportable_payload());
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);

        assert_eq!(started.borrow().len(), 1, "escalation was attempted");
        assert!(tree.active_drag.is_some(), "session kept (no backend)");
        assert!(tree.outbound_drag_source.is_none());
    }

    #[test]
    fn non_exportable_drag_does_not_escalate() {
        let started = std::rc::Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started: started.clone(),
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Plain typed payload, no mime ⇒ not OS-exportable.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(1_u32));
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);

        assert!(started.borrow().is_empty(), "no escalation attempt");
        assert!(tree.active_drag.is_some(), "session unaffected");
    }

    #[test]
    fn in_app_drop_fires_source_on_drag_ended_with_accepted() {
        use crate::drag_payload::DropOutcome;
        use std::cell::Cell;
        use std::rc::Rc;

        let outcome = Rc::new(Cell::new(None));
        let o = outcome.clone();

        let mut tree = WidgetTree::new();
        let source =
            tree.add(FillWidget::new().on_drag_ended(move |outcome, _ctx| o.set(Some(outcome))));
        let _target = tree.add(FillWidget::new().on_drop(|_, _, _| true));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(42_u32));
        tree.collect_from_ctx(ctx, source);

        tree.dispatch_event(WidgetEvent::pointer_up(
            Point::new(150.0, 50.0),
            PointerButton::Primary,
            Modifiers::NONE,
        ));

        assert_eq!(outcome.get(), Some(DropOutcome::InApp { accepted: true }));
    }

    #[test]
    fn escape_fires_source_on_drag_ended_cancelled() {
        use crate::drag_payload::DropOutcome;
        use std::cell::Cell;
        use std::rc::Rc;

        let outcome = Rc::new(Cell::new(None));
        let o = outcome.clone();

        let mut tree = WidgetTree::new();
        let source =
            tree.add(FillWidget::new().on_drag_ended(move |outcome, _ctx| o.set(Some(outcome))));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(0_u32));
        tree.collect_from_ctx(ctx, source);

        tree.press_key(Key::Escape, Modifiers::NONE);
        assert_eq!(outcome.get(), Some(DropOutcome::Cancelled));
    }

    /// Drag out (escalate to OS), then the OS drag re-enters the same window
    /// and drops on an in-app target: the original *typed* payload is
    /// recovered (not lost to the file/text round-trip), and the source's
    /// `on_drag_ended` fires exactly once, with the in-app outcome.
    #[test]
    fn os_drag_reentry_recovers_typed_payload_for_in_app_drop() {
        use crate::drag_payload::{DragPayload, DropOutcome};
        use std::cell::Cell;
        use std::rc::Rc;

        let started = Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started,
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let got_typed = Rc::new(Cell::new(0_u32));
        let ended = Rc::new(Cell::new(0_u32));
        let last_outcome = Rc::new(Cell::new(None));
        let g = got_typed.clone();
        let e = ended.clone();
        let lo = last_outcome.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new().on_drag_ended(move |outcome, _ctx| {
            e.set(e.get() + 1);
            lo.set(Some(outcome));
        }));
        let _target =
            tree.add(
                FillWidget::new().on_drop(move |mut p, _, _| match p.take_typed::<u32>() {
                    Some(v) => {
                        g.set(v);
                        true
                    }
                    None => false,
                }),
            );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // Internal drag with a typed value AND an exportable MIME rep.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            source,
            DragPayload::typed(123_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree.collect_from_ctx(ctx, source);

        // Leave the window → escalate to OS drag (typed payload stashed).
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert!(tree.active_drag.is_none());
        assert!(
            super::has_outbound_typed(),
            "typed payload stashed globally"
        );

        // OS drag re-enters → restored as an internal session with the typed
        // value (not an external file/text drop).
        tree.begin_external_drag(
            Point::new(100.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        let d = tree.active_drag.as_ref().expect("re-entered session");
        assert!(!d.is_external, "re-entry is an internal session");
        assert!(d.payload.has_typed::<u32>(), "typed payload recovered");
        assert_eq!(
            d.payload.text(),
            Some("x"),
            "external view enriched from MIME so DropZone-style targets also accept"
        );
        assert!(!super::has_outbound_typed(), "stash taken by the re-entry");

        // Drop inside on the target → on_drop receives the typed value.
        tree.end_external_drag(
            Point::new(150.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(got_typed.get(), 123, "target received the typed payload");
        assert_eq!(
            ended.get(),
            0,
            "source on_drag_ended not fired by the drop itself"
        );

        // OS posts the terminal event on the source window → exactly one
        // on_drag_ended, and since the drop landed back on the window that
        // started the drag, with the in-app outcome rather than the
        // platform's copy.
        tree.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
        assert_eq!(ended.get(), 1, "on_drag_ended fired exactly once");
        assert_eq!(
            last_outcome.get(),
            Some(DropOutcome::InApp { accepted: true })
        );
    }

    /// The same recovery works across two windows of the same app: window A
    /// starts the drag, the OS drag enters window B, and B's target receives
    /// the original typed payload.
    #[test]
    fn os_drag_reentry_recovers_typed_payload_across_windows() {
        use crate::drag_payload::{DragPayload, DropOutcome};
        use std::cell::Cell;
        use std::rc::Rc;

        let started = Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started,
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        // Window A: starts and escalates.
        let mut tree_a = WidgetTree::new();
        let src = tree_a.add(FillWidget::new());
        tree_a.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            src,
            DragPayload::typed(77_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree_a.collect_from_ctx(ctx, src);
        tree_a.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert!(super::has_outbound_typed());
        assert_eq!(tree_a.outbound_drag_source, Some(src));

        // Window B (separate tree, same thread ⇒ same global stash): the OS
        // drag enters and drops on B's target, which gets the typed value.
        let got = Rc::new(Cell::new(0_u32));
        let g = got.clone();
        let mut tree_b = WidgetTree::new();
        let _t = tree_b.add(FillWidget::new().on_drop(
            move |mut p, _, _| match p.take_typed::<u32>() {
                Some(v) => {
                    g.set(v);
                    true
                }
                None => false,
            },
        ));
        tree_b.layout(SizeProposal::exact(200.0, 100.0));

        tree_b.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(
            tree_b
                .active_drag
                .as_ref()
                .is_some_and(|d| d.payload.has_typed::<u32>()),
            "window B recovered the typed payload"
        );
        tree_b.end_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(
            got.get(),
            77,
            "window B's target received the typed payload"
        );

        // Source window A reports the terminal outcome.
        tree_a.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
    }

    /// A re-entered OS drag that leaves the window again re-stashes the typed
    /// payload (does not start a second OS drag, does not fire on_drag_ended),
    /// so a later window can still recover it.
    #[test]
    fn os_drag_reexit_restashes_payload() {
        use crate::drag_payload::DragPayload;
        use std::rc::Rc;

        let started = Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started: started.clone(),
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            source,
            DragPayload::typed(9_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree.collect_from_ctx(ctx, source);

        // Escalate, then re-enter, then leave again.
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1, "OS drag started once");
        tree.begin_external_drag(
            Point::new(100.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree.os_drag_reentered);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops); // leave again

        assert!(!tree.os_drag_reentered, "re-exited");
        assert!(tree.active_drag.is_none(), "session torn down on re-exit");
        assert!(super::has_outbound_typed(), "payload re-stashed");
        assert_eq!(
            started.borrow().len(),
            1,
            "no second OS drag started on re-exit"
        );
    }

    /// Closing the source window mid-OS-drag clears the global stash, so a
    /// later genuine external drag from another app is NOT misrecovered as the
    /// stale typed payload. (Regression for the CRITICAL stash-leak finding.)
    #[test]
    fn source_window_close_clears_stash_no_hijack() {
        use crate::drag_payload::{DragPayload, ExternalDropData};
        use std::path::PathBuf;
        use std::rc::Rc;

        let started = Rc::new(std::cell::RefCell::new(Vec::new()));
        let mut ops = RecordingWindowOps {
            started,
            started_kinds: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            start: crate::drag_payload::OsDragStart::Started,
            pending_ids: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
            cancels: std::rc::Rc::new(std::cell::Cell::new(0)),
            accepts: std::rc::Rc::new(std::cell::RefCell::new(Vec::new())),
        };

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        let _target = tree.add(FillWidget::new().on_drop(|_, _, _| true));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            source,
            DragPayload::typed(5_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(-5.0, 50.0), &mut ops); // escalate
        assert!(super::has_outbound_typed());
        assert_eq!(tree.outbound_drag_source, Some(source));

        // Window closes mid-drag.
        tree.abort_outbound_drag();
        assert!(
            !super::has_outbound_typed(),
            "stash cleared when the source window closes"
        );
        assert!(tree.outbound_drag_source.is_none());

        // A later real external drag (another app) must present as external,
        // NOT recover the stale typed payload.
        tree.begin_external_drag(
            Point::new(50.0, 50.0),
            ExternalDropData {
                files: vec![PathBuf::from("/tmp/real")],
                ..Default::default()
            },
            &mut ops,
        );
        let d = tree.active_drag.as_ref().expect("external session");
        assert!(
            d.is_external,
            "stale stash did not hijack the new external drag"
        );
        assert!(
            !d.payload.has_typed::<u32>(),
            "no stale typed payload leaked in"
        );
        assert_eq!(d.payload.files(), &[PathBuf::from("/tmp/real")]);
    }

    // ---------------------------------------------------------------
    // P31: the drag's own pointer, outside any sample
    // ---------------------------------------------------------------

    use std::cell::{Cell, RefCell};
    use std::rc::Rc;

    /// A leaf that arms a drag on its own `PointerDown`, the way a real widget's
    /// long-press or `on_drag` handler does.
    ///
    /// The drag HAS to be armed from inside the contact's own dispatch: that is
    /// the only place `current_input` names the finger, so it is the only place
    /// the session can record it. Building the payload from a factory keeps
    /// `DragPayload` (not `Clone`) out of the closure's captured state.
    fn drag_arming_leaf(
        slot: Rc<Cell<Option<WidgetId>>>,
        payload: impl Fn() -> crate::drag_payload::DragPayload + 'static,
    ) -> impl crate::widget::Widget {
        let armed = Cell::new(false);
        FillWidget::new().on_pointer_event(move |event, ctx| {
            if matches!(event, crate::event::WidgetEvent::PointerDown { .. })
                && !armed.replace(true)
                && let Some(id) = slot.get()
            {
                ctx.start_drag(id, payload());
            }
            crate::event::EventResponse::Ignored
        })
    }

    /// Press a fresh touch contact at `at`, returning its id.
    fn touch_press(
        tree: &mut WidgetTree,
        at: Point,
        ops: &mut dyn crate::window::WindowOps,
    ) -> crate::pointer::PointerId {
        use crate::pointer::{
            BackendDeviceKey, PointerIdAllocator, PointerInfo, PointerPhase, PointerSample,
        };
        use std::sync::atomic::{AtomicU64, Ordering};
        static NEXT: AtomicU64 = AtomicU64::new(91_000);
        let contact = PointerIdAllocator::global().begin(
            BackendDeviceKey::DEFAULT,
            NEXT.fetch_add(1, Ordering::Relaxed),
        );
        tree.dispatch_pointer_with_ops(
            PointerSample {
                pointer: PointerInfo::touch(contact, crate::pointer::EventTime::from_millis(1)),
                phase: PointerPhase::Down,
                position: at,
                button: None,
                modifiers: Modifiers::NONE,
                coalesced: Vec::new(),
            },
            ops,
        );
        contact
    }

    /// The placement an overlay currently carries.
    fn placement_of(
        tree: &WidgetTree,
        id: crate::overlay::OverlayId,
    ) -> crate::overlay::OverlayPlacement {
        tree.overlay_manager
            .stack
            .iter()
            .find(|o| o.id == id)
            .map(|o| o.placement.clone())
            .expect("the overlay is in the stack")
    }

    /// The drag tick fires from `layout()`, outside any sample — and it must
    /// still tell its handler which device is dragging.
    ///
    /// This is the whole of the coarse auto-scroll band's reachability: every
    /// data view's `on_drag_tick` asks `ctx.pointer_kind()` for the band, and
    /// before the session carried a pointer the answer was `Mouse` for the whole
    /// of a finger drag, so the wider band could never apply.
    #[test]
    fn a_drag_tick_reports_the_device_that_started_the_drag() {
        let seen: Rc<RefCell<Vec<teksilo_tokens::PointerKind>>> = Rc::new(RefCell::new(Vec::new()));
        let s = seen.clone();

        let mut tree = WidgetTree::new();
        let target = tree.add(
            FillWidget::new()
                .on_drop(|_, _, _| true)
                .on_drag_tick(move |_pos, ctx| s.borrow_mut().push(ctx.pointer_kind())),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        // A mouse drag first: the pre-existing answer, unchanged.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(target, crate::drag_payload::DragPayload::typed(1_u8));
        tree.collect_from_ctx(ctx, target);
        tree.handle_drag_move(Point::new(50.0, 50.0), &mut crate::window::NoopWindowOps);
        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert_eq!(
            *seen.borrow(),
            vec![teksilo_tokens::PointerKind::Mouse],
            "a mouse drag still reports the mouse"
        );
        tree.cancel_active_drag(&mut crate::window::NoopWindowOps);
        seen.borrow_mut().clear();

        // Now a finger, arming its drag from inside its own press. The same node
        // is source, drop target and tick owner — a row of a reorderable list.
        let mut ops = crate::window::NoopWindowOps;
        let s = seen.clone();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let armed = Cell::new(false);
        let s2 = slot.clone();
        let mut tree = WidgetTree::new();
        let row = tree.add(
            FillWidget::new()
                .on_pointer_event(move |event, ctx| {
                    if matches!(event, crate::event::WidgetEvent::PointerDown { .. })
                        && !armed.replace(true)
                        && let Some(id) = s2.get()
                    {
                        ctx.start_drag(id, crate::drag_payload::DragPayload::typed(2_u8));
                    }
                    crate::event::EventResponse::Ignored
                })
                .on_drop(|_, _, _| true)
                .on_drag_tick(move |_pos, ctx| s.borrow_mut().push(ctx.pointer_kind())),
        );
        slot.set(Some(row));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        touch_press(&mut tree, Point::new(50.0, 50.0), &mut ops);
        assert!(tree.active_drag.is_some(), "the finger armed a drag");
        tree.handle_drag_move(Point::new(50.0, 50.0), &mut ops);
        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert_eq!(
            *seen.borrow(),
            vec![teksilo_tokens::PointerKind::Touch],
            "and a finger drag reports the finger, from a tick that has no sample"
        );
    }

    /// The preview is placed clear of a coarse contact and byte-identically at
    /// the point for a mouse.
    ///
    /// A preview pinned to the pixel a finger reported sits under the hand
    /// carrying it, so the user drags something they cannot see. `AtPointer`
    /// stays exactly `AtPointer` for a cursor, which is what keeps mouse
    /// placement unchanged.
    #[test]
    fn the_preview_avoids_a_coarse_contact_and_still_pins_a_cursor() {
        use crate::overlay::OverlayPlacement;

        let mut ops = crate::window::NoopWindowOps;
        let at = Point::new(60.0, 40.0);

        // Mouse.
        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag_with_preview(
            source,
            crate::drag_payload::DragPayload::typed(1_u8),
            Box::new(FillWidget::new()),
        );
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(at, &mut ops);
        let overlay = tree
            .active_drag
            .as_ref()
            .and_then(|d| d.preview_overlay_id)
            .expect("the preview overlay exists");
        assert!(
            matches!(placement_of(&tree, overlay), OverlayPlacement::AtPointer(p) if p == at),
            "a mouse keeps AtPointer at the reported point, unchanged",
        );

        // Finger: the same drag, armed from inside a contact's press so the
        // session records it, with a preview attached the way the router does.
        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let source = tree.add(drag_arming_leaf_with_preview(slot.clone()));
        slot.set(Some(source));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        touch_press(&mut tree, at, &mut ops);
        assert_eq!(
            tree.active_drag.as_ref().map(|d| d.pointer.kind),
            Some(teksilo_tokens::PointerKind::Touch),
            "the session recorded the finger",
        );
        tree.handle_drag_move(at, &mut ops);
        let overlay = tree
            .active_drag
            .as_ref()
            .and_then(|d| d.preview_overlay_id)
            .expect("the preview overlay exists");
        match placement_of(&tree, overlay) {
            OverlayPlacement::AtPointerAvoiding { point, avoid } => {
                assert_eq!(point, at);
                assert!(
                    avoid.contains(at),
                    "the rectangle to clear is centred on the contact, so no \
                     placement that honours it can put the preview under the hand",
                );
            }
            other => panic!("a coarse pointer must avoid its own contact, got {other:?}"),
        }
    }

    /// The `drag_arming_leaf` above, but with a preview — the shape
    /// `ListView`/`TreeView` use.
    fn drag_arming_leaf_with_preview(
        slot: Rc<Cell<Option<WidgetId>>>,
    ) -> impl crate::widget::Widget {
        let armed = Cell::new(false);
        FillWidget::new().on_pointer_event(move |event, ctx| {
            if matches!(event, crate::event::WidgetEvent::PointerDown { .. })
                && !armed.replace(true)
                && let Some(id) = slot.get()
            {
                ctx.start_drag_with_preview(
                    id,
                    crate::drag_payload::DragPayload::typed(1_u8),
                    Box::new(FillWidget::new()),
                );
            }
            crate::event::EventResponse::Ignored
        })
    }

    /// Escalation revokes the pointer that was **dragging**, not whichever
    /// pointer the singular accessor happens to name.
    ///
    /// A drag tick can move the reported position outside the window (it
    /// scrolls the content under a stationary finger), and a tick runs outside
    /// any sample — where `current_pointer_id` answers "the mouse". Cancelling
    /// the mouse there would leave the real contact armed with a sequence for a
    /// drag the OS had taken over.
    #[test]
    fn an_os_drag_started_by_a_finger_cancels_that_finger() {
        let cancelled: Rc<RefCell<Vec<(crate::pointer::PointerId, crate::pointer::CancelReason)>>> =
            Rc::new(RefCell::new(Vec::new()));
        let c = cancelled.clone();

        let mut ops = RecordingWindowOps::new(true);
        let kinds = ops.started_kinds.clone();

        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let armed = Cell::new(false);
        let s2 = slot.clone();
        let source = tree.add(
            FillWidget::new()
                .on_pointer_event(move |event, ctx| {
                    if matches!(event, crate::event::WidgetEvent::PointerDown { .. })
                        && !armed.replace(true)
                        && let Some(id) = s2.get()
                    {
                        ctx.start_drag(id, exportable_payload());
                    }
                    crate::event::EventResponse::Ignored
                })
                .on_pointer_cancel(move |pointer, reason, _ctx| {
                    c.borrow_mut().push((pointer.id, reason));
                }),
        );
        slot.set(Some(source));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let contact = touch_press(&mut tree, Point::new(20.0, 50.0), &mut ops);
        assert!(tree.active_drag.is_some());

        // Out of the window: the drag escalates.
        tree.handle_drag_move(Point::new(-40.0, 50.0), &mut ops);
        assert_eq!(tree.outbound_drag_source, Some(source));
        assert_eq!(
            *kinds.borrow(),
            vec![teksilo_tokens::PointerKind::Touch],
            "the platform is told which device is dragging — Wayland needs the \
             touch-down serial, not a button serial"
        );
        assert_eq!(
            *cancelled.borrow(),
            vec![(contact, crate::pointer::CancelReason::OsDragStarted)],
            "the finger is the pointer revoked"
        );
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }

    /// An Escape-cancelled finger drag still reports the finger to the source.
    ///
    /// The Escape arrives as a key dispatch, which serves no pointer at all, so
    /// without the drag's own pointer installed the source's `on_drag_ended`
    /// would be told a mouse cancelled the drag a finger had been carrying —
    /// the same divergence as the tick, at the other end of the same drag.
    #[test]
    fn an_escape_cancelled_finger_drag_reports_the_finger_to_its_source() {
        let seen: Rc<RefCell<Vec<teksilo_tokens::PointerKind>>> = Rc::new(RefCell::new(Vec::new()));
        let s = seen.clone();

        let mut ops = crate::window::NoopWindowOps;
        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let armed = Cell::new(false);
        let s2 = slot.clone();
        let source = tree.add(
            FillWidget::new()
                .on_pointer_event(move |event, ctx| {
                    if matches!(event, crate::event::WidgetEvent::PointerDown { .. })
                        && !armed.replace(true)
                        && let Some(id) = s2.get()
                    {
                        ctx.start_drag(id, crate::drag_payload::DragPayload::typed(1_u8));
                    }
                    crate::event::EventResponse::Ignored
                })
                .on_drag_ended(move |_outcome, ctx| s.borrow_mut().push(ctx.pointer_kind())),
        );
        slot.set(Some(source));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        touch_press(&mut tree, Point::new(50.0, 50.0), &mut ops);
        assert!(tree.active_drag.is_some());
        tree.dispatch_event_with_ops(
            crate::event::WidgetEvent::KeyDown {
                key: crate::event::Key::Escape,
                modifiers: crate::event::Modifiers::NONE,
                text: None,
            },
            &mut ops,
        );
        assert_eq!(*seen.borrow(), vec![teksilo_tokens::PointerKind::Touch]);
    }

    /// The widget's verdict reaches the platform, and only when it changes.
    ///
    /// An inbound backend has to answer the drag source before the tree has seen
    /// the position, so its first answer is about formats alone; without this the
    /// OS showed "will accept" over a target that refuses the payload.
    #[test]
    fn the_accept_setter_receives_the_widget_verdict() {
        use crate::drag_state::DropFeedback;

        let verdict = Rc::new(Cell::new(false));
        let v = verdict.clone();
        let mut ops = RecordingWindowOps::new(true);
        let accepts = ops.accepts.clone();

        let mut tree = WidgetTree::new();
        tree.add(
            FillWidget::new()
                .on_drag_hover(move |_p, _pos, _ctx| {
                    if v.get() {
                        DropFeedback::Accept
                    } else {
                        DropFeedback::NoFeedback
                    }
                })
                .on_drop(|_, _, _| true),
        );
        tree.layout(SizeProposal::exact(200.0, 100.0));

        tree.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData {
                files: vec![std::path::PathBuf::from("/tmp/a.png")],
                ..Default::default()
            },
            &mut ops,
        );
        assert_eq!(
            *accepts.borrow(),
            vec![false],
            "a refusing target is reported to the OS as a refusal"
        );

        // Same answer again: nothing more is pushed. The OS side is a round trip
        // per call and a motion stream would repeat it every sample.
        tree.update_external_drag(Point::new(52.0, 50.0), &mut ops);
        assert_eq!(
            *accepts.borrow(),
            vec![false],
            "an unchanged answer is not re-sent"
        );

        // The target changes its mind.
        verdict.set(true);
        tree.update_external_drag(Point::new(54.0, 50.0), &mut ops);
        assert_eq!(
            *accepts.borrow(),
            vec![false, true],
            "and a change is pushed once"
        );
    }

    /// An in-app drag is not an OS drag, and must not push an accept state to a
    /// platform that has nothing in flight to revise.
    #[test]
    fn an_in_app_drag_pushes_no_os_accept_state() {
        let mut ops = RecordingWindowOps::new(true);
        let accepts = ops.accepts.clone();

        let mut tree = WidgetTree::new();
        let source = tree.add(FillWidget::new());
        tree.add(FillWidget::new().on_drop(|_, _, _| true));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, crate::drag_payload::DragPayload::typed(1_u8));
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_move(Point::new(50.0, 50.0), &mut ops);

        assert!(accepts.borrow().is_empty());
    }

    /// A cancelled OS drag tears the source down **exactly once**, whichever
    /// order the platform reports it in.
    ///
    /// Two independent paths could fire the source's `on_drag_ended`: the
    /// terminal `DragEnded` on the window that started the drag, and the abort
    /// delivered to whichever window was holding the re-entered session. Only
    /// the first owns it, and a backend that reports a terminal twice (some
    /// compositors send both `dnd_finished` and `cancelled`) must not double it.
    #[test]
    fn a_cancelled_os_drag_tears_the_source_down_exactly_once() {
        use crate::drag_payload::{DragPayload, DropOutcome};

        let ended = Rc::new(RefCell::new(Vec::new()));
        let e = ended.clone();
        let mut ops = RecordingWindowOps::new(true);

        // Window A starts and escalates.
        let mut tree_a = WidgetTree::new();
        let src = tree_a.add(
            FillWidget::new().on_drag_ended(move |outcome, _ctx| e.borrow_mut().push(outcome)),
        );
        tree_a.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            src,
            DragPayload::typed(5_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree_a.collect_from_ctx(ctx, src);
        tree_a.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert!(super::outbound_is_live());

        // Window B picks the drag up as a re-entered session.
        let left = Rc::new(Cell::new(0_u32));
        let l = left.clone();
        let mut tree_b = WidgetTree::new();
        tree_b.add(
            FillWidget::new()
                .on_drag_hover(|_, _, _| crate::drag_state::DropFeedback::Accept)
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree_b.layout(SizeProposal::exact(200.0, 100.0));
        let accepts = ops.accepts.clone();
        accepts.borrow_mut().clear();
        tree_b.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree_b.os_drag_reentered);
        assert_eq!(
            *accepts.borrow(),
            vec![true],
            "a re-entered app drag still negotiates with the OS: its offer is \
             live and a refusal must still reach the compositor's cursor",
        );

        // The OS aborts. B is told, and clears without claiming the source's
        // notification — B has no source widget.
        tree_b.abort_external_drag(&mut ops);
        assert!(tree_b.active_drag.is_none(), "B's session is gone");
        assert!(!tree_b.os_drag_reentered);
        assert_eq!(left.get(), 1, "B's highlighted target was cleared");
        assert!(
            ended.borrow().is_empty(),
            "the abort must not fire the source's on_drag_ended — the source \
             window owns that"
        );

        // A's terminal event fires it, once.
        tree_a.handle_os_drag_ended(DropOutcome::Cancelled, &mut ops);
        assert_eq!(*ended.borrow(), vec![DropOutcome::Cancelled]);

        // A second terminal from a backend that reports both must add nothing.
        tree_a.handle_os_drag_ended(DropOutcome::Cancelled, &mut ops);
        assert_eq!(
            *ended.borrow(),
            vec![DropOutcome::Cancelled],
            "exactly once"
        );
    }

    /// A window holding a re-entered OS drag notices when the drag ends
    /// elsewhere, and drops the session on its next layout pass.
    ///
    /// The terminal `DragEnded` goes to the window that *started* the drag, and
    /// that is not necessarily the one showing the re-entered session — so
    /// without this the other window kept a live `active_drag`, a highlighted
    /// drop target and an `os_drag_reentered` flag for a drag that no longer
    /// existed, for the rest of the process. Nothing further ever arrives for
    /// it from the OS, so the condition has to be noticed from the inside.
    #[test]
    fn a_reentered_session_is_reaped_when_the_os_drag_ends_elsewhere() {
        use crate::drag_payload::{DragPayload, DropOutcome};

        let mut ops = RecordingWindowOps::new(true);

        let mut tree_a = WidgetTree::new();
        let src = tree_a.add(FillWidget::new());
        tree_a.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(
            src,
            DragPayload::typed(9_u32).with_mime("text/plain", b"x".to_vec()),
        );
        tree_a.collect_from_ctx(ctx, src);
        tree_a.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);

        let left = Rc::new(Cell::new(0_u32));
        let l = left.clone();
        let mut tree_b = WidgetTree::new();
        tree_b.add(
            FillWidget::new()
                .on_drag_hover(|_, _, _| crate::drag_state::DropFeedback::Accept)
                .on_drag_leave(move |_ctx| l.set(l.get() + 1))
                .on_drop(|_, _, _| true),
        );
        tree_b.layout(SizeProposal::exact(200.0, 100.0));
        tree_b.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree_b.active_drag.is_some() && tree_b.os_drag_reentered);

        // A's window reports the terminal outcome. B hears nothing.
        tree_a.handle_os_drag_ended(DropOutcome::Cancelled, &mut ops);
        assert!(
            tree_b.active_drag.is_some(),
            "B has not been told anything yet"
        );

        // B's next layout pass notices the stash is dead.
        tree_b.layout(SizeProposal::exact(200.0, 100.0));
        assert!(tree_b.active_drag.is_none(), "the dead session was reaped");
        assert!(!tree_b.os_drag_reentered);
        assert_eq!(left.get(), 1, "and its highlighted target was cleared");
    }

    /// A re-entered app drag carries the device that started it, so the window
    /// it lands in reads the finger — the one case where an inbound OS drag's
    /// kind is knowable at all.
    #[test]
    fn a_reentered_app_drag_recovers_the_device_that_started_it() {
        let mut ops = RecordingWindowOps::new(true);

        let mut tree_a = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let src = tree_a.add(drag_arming_leaf(slot.clone(), exportable_payload));
        slot.set(Some(src));
        tree_a.layout(SizeProposal::exact(200.0, 100.0));
        touch_press(&mut tree_a, Point::new(20.0, 50.0), &mut ops);
        tree_a.handle_drag_move(Point::new(-5.0, 50.0), &mut ops);
        assert!(super::outbound_is_live());

        let mut tree_b = WidgetTree::new();
        tree_b.add(FillWidget::new().on_drop(|_, _, _| true));
        tree_b.layout(SizeProposal::exact(200.0, 100.0));
        tree_b.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(
            tree_b.active_drag.as_ref().map(|d| d.pointer.kind),
            Some(teksilo_tokens::PointerKind::Touch),
        );

        // A foreign drag, by contrast, is credited to no device at all.
        let mut tree_c = WidgetTree::new();
        tree_c.add(FillWidget::new().on_drop(|_, _, _| true));
        tree_c.layout(SizeProposal::exact(200.0, 100.0));
        tree_a.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
        tree_c.begin_external_drag(
            Point::new(50.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(
            tree_c.active_drag.as_ref().map(|d| d.pointer.kind),
            Some(teksilo_tokens::PointerKind::Unknown),
            "no OS names the source's device to a destination",
        );
    }

    /// A re-stash that races in *after* the drag's terminal event must not
    /// resurrect a finished drag (cross-window drop-on-nothing race). Tests the
    /// liveness gate directly. (Regression for the HIGH race finding.)
    #[test]
    fn restash_after_drag_ended_is_noop() {
        use crate::drag_payload::DragPayload;

        super::outbound_begin(
            DragPayload::typed(1_u32).with_mime("text/plain", b"x".to_vec()),
            crate::pointer::PointerInfo::mouse(crate::pointer::EventTime::ZERO),
        );
        assert!(super::has_outbound_typed());
        // A window re-entered and took the payload.
        let held = super::outbound_take_if_live().expect("payload taken while live");
        assert!(!super::has_outbound_typed());
        // The source window's terminal DragEnded ends the drag first.
        super::outbound_end();
        // The other window's late re-stash must be dropped, not resurrected.
        super::outbound_restash(held);
        assert!(
            !super::has_outbound_typed(),
            "ended drag is not resurrected by a racing re-stash"
        );
        // And a take after end yields nothing.
        assert!(super::outbound_take_if_live().is_none());
    }

    // ---------------------------------------------------------------
    // OsHandoff::Eager and the two-phase native start
    // ---------------------------------------------------------------

    fn pending() -> crate::drag_payload::OsDragStart {
        crate::drag_payload::OsDragStart::Pending(crate::drag_payload::OsDragId::next())
    }

    /// The id of the last start `ops` answered `Pending`.
    fn last_pending(ops: &RecordingWindowOps) -> crate::drag_payload::OsDragId {
        *ops.pending_ids.borrow().last().expect("a pending start")
    }

    fn eager_payload() -> crate::drag_payload::DragPayload {
        exportable_payload().with_os_handoff(crate::drag_payload::OsHandoff::Eager)
    }

    /// A tree with an exportable drag in flight from `source`, which records
    /// every `on_drag_ended` outcome in `ended`.
    fn tree_dragging(
        payload: crate::drag_payload::DragPayload,
    ) -> (
        WidgetTree,
        WidgetId,
        Rc<RefCell<Vec<crate::drag_payload::DropOutcome>>>,
    ) {
        let ended: Rc<RefCell<Vec<crate::drag_payload::DropOutcome>>> =
            Rc::new(RefCell::new(Vec::new()));
        let e = ended.clone();
        let mut tree = WidgetTree::new();
        let source =
            tree.add(FillWidget::new().on_drag_ended(move |o, _ctx| e.borrow_mut().push(o)));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, payload);
        tree.collect_from_ctx(ctx, source);
        assert!(tree.active_drag.is_some());
        (tree, source, ended)
    }

    #[test]
    fn an_eager_drag_hands_off_on_the_first_pointer_move_inside_the_window() {
        let mut ops = RecordingWindowOps::new(true);
        let started = ops.started.clone();
        let (mut tree, source, ended) = tree_dragging(eager_payload());

        // A real pointer sample, through the router, well inside the window.
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(100.0, 50.0)), &mut ops);
        assert_eq!(started.borrow().len(), 1, "begin_os_drag asked once");
        assert!(tree.active_drag.is_none(), "in-app session handed over");
        assert_eq!(tree.outbound_drag_source, Some(source));
        assert!(super::has_outbound_typed(), "typed payload parked");

        // Further moves are the OS's; nothing asks again.
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(120.0, 50.0)), &mut ops);
        assert_eq!(started.borrow().len(), 1);
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::OsCopy, &mut ops);
        assert_eq!(
            *ended.borrow(),
            vec![crate::drag_payload::DropOutcome::OsCopy]
        );
    }

    #[test]
    fn an_at_window_edge_drag_stays_in_app_while_the_pointer_is_inside() {
        let mut ops = RecordingWindowOps::new(true);
        let started = ops.started.clone();
        let (mut tree, source, _ended) = tree_dragging(exportable_payload());
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(100.0, 50.0)), &mut ops);
        assert!(started.borrow().is_empty());
        assert!(tree.active_drag.is_some());
        tree.handle_drag_sample(Point::new(-5.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1, "the window edge still hands off");
        assert_eq!(tree.outbound_drag_source, Some(source));
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }

    #[test]
    fn an_eager_drag_waits_for_a_pointer_sample_rather_than_a_drag_tick() {
        let mut ops = RecordingWindowOps::new(true);
        let started = ops.started.clone();
        let (mut tree, _source, _ended) = tree_dragging(eager_payload());

        // A drag tick (and every other move that is not a sample) runs outside
        // any input event; macOS cannot start a drag there.
        tree.handle_drag_move(Point::new(100.0, 50.0), &mut ops);
        tree.layout(SizeProposal::exact(200.0, 100.0));
        assert!(started.borrow().is_empty(), "no handoff from a tick");
        assert!(tree.active_drag.is_some());

        tree.handle_drag_sample(Point::new(101.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1, "the next sample hands off");
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }

    #[test]
    fn a_refused_eager_handoff_is_asked_again_on_the_next_sample() {
        let mut ops = RecordingWindowOps::new(false);
        let started = ops.started.clone();
        let (mut tree, source, _ended) = tree_dragging(eager_payload());

        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1);
        assert!(
            tree.active_drag.is_some(),
            "a refusal keeps the in-app drag"
        );
        tree.handle_drag_sample(Point::new(101.0, 50.0), &mut ops);
        assert_eq!(
            started.borrow().len(),
            2,
            "no latch: the next move asks again"
        );

        ops.start = crate::drag_payload::OsDragStart::Started;
        tree.handle_drag_sample(Point::new(102.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 3);
        assert_eq!(tree.outbound_drag_source, Some(source), "handed off once");
        tree.handle_drag_sample(Point::new(103.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 3);
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }

    #[test]
    fn eager_without_anything_to_export_never_hands_off() {
        let mut ops = RecordingWindowOps::new(true);
        let started = ops.started.clone();
        let (mut tree, _source, _ended) = tree_dragging(
            crate::drag_payload::DragPayload::typed(1_u32)
                .with_os_handoff(crate::drag_payload::OsHandoff::Eager),
        );
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        tree.handle_drag_sample(Point::new(-5.0, 50.0), &mut ops);
        assert!(started.borrow().is_empty());
        assert!(tree.active_drag.is_some());

        // An inbound drag from another application never escalates either.
        let mut tree = WidgetTree::new();
        tree.add(FillWidget::new().on_drop(|_, _, _| true));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let data = crate::drag_payload::ExternalDropData {
            text: Some("x".into()),
            ..Default::default()
        };
        tree.begin_external_drag(Point::new(10.0, 10.0), data, &mut ops);
        tree.update_external_drag(Point::new(-5.0, 10.0), &mut ops);
        assert!(started.borrow().is_empty());
    }

    #[test]
    fn a_pending_start_keeps_the_in_app_drag_until_the_platform_confirms_it() {
        let mut ops = RecordingWindowOps::answering(pending());
        let started = ops.started.clone();
        let (mut tree, source, ended) = tree_dragging(eager_payload());

        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1);
        assert!(
            tree.active_drag.is_some(),
            "the session waits for the platform"
        );
        assert!(tree.outbound_drag_source.is_none());
        assert!(
            !super::has_outbound_typed(),
            "nothing parked before the start"
        );

        // A start is in flight: more moves keep the in-app drag going and do not
        // ask a second time.
        tree.handle_drag_sample(Point::new(110.0, 50.0), &mut ops);
        tree.handle_drag_sample(Point::new(-5.0, 50.0), &mut ops);
        assert_eq!(started.borrow().len(), 1, "one request per start");
        assert!(tree.active_drag.is_some());

        tree.confirm_os_drag(last_pending(&ops), &mut ops);
        assert!(
            tree.active_drag.is_none(),
            "handed over at the confirmation"
        );
        assert_eq!(tree.outbound_drag_source, Some(source));
        assert!(super::has_outbound_typed());
        assert!(ended.borrow().is_empty());
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::OsCopy, &mut ops);
        assert_eq!(
            *ended.borrow(),
            vec![crate::drag_payload::DropOutcome::OsCopy]
        );
    }

    #[test]
    fn a_release_before_the_native_drag_starts_ends_the_drag_as_cancelled() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::answering(pending());
        let cancels = ops.cancels.clone();
        let dropped = Rc::new(Cell::new(0_u32));
        let d = dropped.clone();
        let (mut tree, _source, ended) = tree_dragging(eager_payload());
        tree.add(FillWidget::new().on_drop(move |_, _, _| {
            d.set(d.get() + 1);
            true
        }));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        tree.handle_drag_sample(Point::new(150.0, 50.0), &mut ops);
        tree.dispatch_event_with_ops(
            WidgetEvent::pointer_up(
                Point::new(150.0, 50.0),
                PointerButton::Primary,
                Modifiers::NONE,
            ),
            &mut ops,
        );
        assert!(tree.active_drag.is_none());
        assert_eq!(dropped.get(), 0, "no drop is delivered");
        assert_eq!(*ended.borrow(), vec![DropOutcome::Cancelled]);
        assert_eq!(cancels.get(), 1, "the platform is told to drop the start");

        // The platform's late confirmation finds nothing waiting and asks it to
        // end the native drag nothing owns; its terminal event is silent.
        tree.confirm_os_drag(last_pending(&ops), &mut ops);
        assert_eq!(cancels.get(), 2);
        assert!(tree.outbound_drag_source.is_none());
        tree.handle_os_drag_ended(DropOutcome::Cancelled, &mut ops);
        assert_eq!(ended.borrow().len(), 1, "on_drag_ended fired once");
        tree.assert_no_leaked_pointer_state();
    }

    #[test]
    fn escape_before_the_native_drag_starts_abandons_the_start() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::answering(pending());
        let cancels = ops.cancels.clone();
        let (mut tree, _source, ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        tree.dispatch_event_with_ops(
            WidgetEvent::KeyDown {
                key: Key::Escape,
                modifiers: Modifiers::NONE,
                text: None,
            },
            &mut ops,
        );
        assert_eq!(cancels.get(), 1);
        assert!(tree.os_drag_pending.is_none());
        assert_eq!(*ended.borrow(), vec![DropOutcome::Cancelled]);
    }

    #[test]
    fn a_start_that_fails_on_the_platform_ends_the_drag_once() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::answering(pending());
        let cancels = ops.cancels.clone();
        let (mut tree, _source, ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);

        // No press serial, no selection ownership: the backend reports the
        // drag ended without starting it.
        tree.handle_os_drag_ended(DropOutcome::Cancelled, &mut ops);
        assert!(tree.active_drag.is_none());
        assert!(tree.os_drag_pending.is_none());
        assert_eq!(*ended.borrow(), vec![DropOutcome::Cancelled]);
        assert_eq!(cancels.get(), 0, "the backend ended it already");
    }

    #[test]
    fn closing_the_window_drops_a_pending_start() {
        let mut ops = RecordingWindowOps::answering(pending());
        let (mut tree, _source, _ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        assert!(tree.os_drag_pending.is_some());
        tree.abort_outbound_drag();
        assert!(tree.os_drag_pending.is_none());
        assert!(!super::outbound_is_live());
    }

    #[test]
    fn an_inbound_drag_before_the_confirmation_is_the_native_drag_itself() {
        let mut ops = RecordingWindowOps::answering(pending());
        let (mut tree, source, _ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);

        tree.begin_external_drag(
            Point::new(100.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(tree.outbound_drag_source, Some(source), "handed over first");
        assert!(tree.os_drag_reentered, "then re-entered as our own drag");
        assert!(
            tree.active_drag
                .as_ref()
                .is_some_and(|d| d.payload.has_typed::<u32>()),
            "with the typed payload recovered"
        );
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }

    /// The reported case: a file manager lying wholly inside the source window.
    /// The compositor sends this window `leave` while the pointer is still
    /// inside its rectangle, and that `Left` must park the payload again
    /// rather than end the drag, so the destination's drop completes it.
    #[test]
    fn a_leave_while_the_pointer_is_inside_the_window_keeps_the_os_drag_alive() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::new(true);
        let (mut tree, _source, ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        tree.begin_external_drag(
            Point::new(100.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree.os_drag_reentered);

        tree.update_external_drag(Point::new(110.0, 50.0), &mut ops);
        tree.cancel_external_drag(&mut ops);
        assert!(!tree.os_drag_reentered);
        assert!(tree.active_drag.is_none());
        assert!(super::has_outbound_typed(), "payload parked again");
        assert!(ended.borrow().is_empty(), "the drag is not over");

        tree.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
        assert_eq!(*ended.borrow(), vec![DropOutcome::OsCopy]);
    }

    #[test]
    fn an_eager_drag_dropped_back_on_its_own_window_reports_the_in_app_outcome() {
        use crate::drag_payload::DropOutcome;
        for accepts in [true, false] {
            let mut ops = RecordingWindowOps::new(true);
            let (mut tree, _source, ended) = tree_dragging(eager_payload());
            tree.add(FillWidget::new().on_drop(move |_, _, _| accepts));
            tree.layout(SizeProposal::exact(200.0, 100.0));

            tree.handle_drag_sample(Point::new(150.0, 50.0), &mut ops);
            tree.begin_external_drag(
                Point::new(150.0, 50.0),
                crate::drag_payload::ExternalDropData::default(),
                &mut ops,
            );
            tree.end_external_drag(
                Point::new(150.0, 50.0),
                crate::drag_payload::ExternalDropData::default(),
                &mut ops,
            );
            assert!(
                ended.borrow().is_empty(),
                "the platform still has to end it"
            );
            // Every backend advertises copy only, so the platform says copy.
            tree.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
            assert_eq!(
                *ended.borrow(),
                vec![DropOutcome::InApp { accepted: accepts }]
            );
        }

        // A drag that came back and left again was not dropped here: the
        // platform's outcome stands.
        let mut ops = RecordingWindowOps::new(true);
        let (mut tree, _source, ended) = tree_dragging(eager_payload());
        tree.handle_drag_sample(Point::new(150.0, 50.0), &mut ops);
        tree.begin_external_drag(
            Point::new(150.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        tree.cancel_external_drag(&mut ops);
        tree.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
        assert_eq!(*ended.borrow(), vec![DropOutcome::OsCopy]);
    }

    #[test]
    fn an_eager_drag_asks_the_platform_once_however_often_it_comes_back() {
        let mut ops = RecordingWindowOps::new(true);
        let started = ops.started.clone();
        for round in 1..=2 {
            let (mut tree, _source, _ended) = tree_dragging(eager_payload());
            tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
            for _ in 0..2 {
                tree.begin_external_drag(
                    Point::new(100.0, 50.0),
                    crate::drag_payload::ExternalDropData::default(),
                    &mut ops,
                );
                tree.update_external_drag(Point::new(-5.0, 50.0), &mut ops);
                assert!(!tree.os_drag_reentered, "re-exited");
            }
            assert_eq!(started.borrow().len(), round, "one start per drag");
            tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
        }
    }

    #[test]
    fn an_eager_finger_drag_tells_the_platform_it_is_a_finger_and_revokes_it() {
        let cancelled: Rc<RefCell<Vec<(crate::pointer::PointerId, crate::pointer::CancelReason)>>> =
            Rc::new(RefCell::new(Vec::new()));
        let c = cancelled.clone();
        let mut ops = RecordingWindowOps::answering(pending());
        let kinds = ops.started_kinds.clone();

        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let source = tree.add(
            drag_arming_leaf(slot.clone(), eager_payload).on_pointer_cancel(
                move |pointer, reason, _ctx| {
                    c.borrow_mut().push((pointer.id, reason));
                },
            ),
        );
        slot.set(Some(source));
        tree.layout(SizeProposal::exact(200.0, 100.0));
        let contact = touch_press(&mut tree, Point::new(20.0, 50.0), &mut ops);
        assert!(tree.active_drag.is_some());

        tree.dispatch_pointer_with_ops(
            crate::pointer::PointerSample {
                pointer: crate::pointer::PointerInfo::touch(
                    contact,
                    crate::pointer::EventTime::from_millis(30),
                ),
                phase: crate::pointer::PointerPhase::Move,
                position: Point::new(40.0, 50.0),
                button: None,
                modifiers: Modifiers::NONE,
                coalesced: Vec::new(),
            },
            &mut ops,
        );
        assert_eq!(*kinds.borrow(), vec![teksilo_tokens::PointerKind::Touch]);
        assert!(
            cancelled.borrow().is_empty(),
            "nothing revoked before the start"
        );

        // The confirmation arrives from the platform, outside any sample.
        tree.confirm_os_drag(last_pending(&ops), &mut ops);
        assert_eq!(
            *cancelled.borrow(),
            vec![(contact, crate::pointer::CancelReason::OsDragStarted)]
        );
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
        tree.assert_no_leaked_pointer_state();
    }

    /// A late start or end of a drag released before its start is not
    /// credited to the next drag, which is waiting for its own start.
    #[test]
    fn a_late_event_of_an_earlier_drag_does_not_touch_the_next_one() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::answering(pending());
        let cancels = ops.cancels.clone();
        let (mut tree, source, ended) = tree_dragging(eager_payload());

        // Drag 1 asks, and is released before its start.
        tree.handle_drag_sample(Point::new(100.0, 50.0), &mut ops);
        let first = last_pending(&ops);
        tree.dispatch_event_with_ops(
            WidgetEvent::pointer_up(
                Point::new(100.0, 50.0),
                PointerButton::Primary,
                Modifiers::NONE,
            ),
            &mut ops,
        );
        assert_eq!(*ended.borrow(), vec![DropOutcome::Cancelled]);
        assert_eq!(cancels.get(), 1);

        // Drag 2 asks.
        let mut ctx = crate::widget::EventContext::new();
        ctx.start_drag(source, eager_payload());
        tree.collect_from_ctx(ctx, source);
        tree.handle_drag_sample(Point::new(110.0, 50.0), &mut ops);
        let second = last_pending(&ops);
        assert_ne!(first, second);

        // Drag 1's late events change nothing for drag 2.
        tree.confirm_os_drag(first, &mut ops);
        tree.handle_os_drag_ended_for(Some(first), DropOutcome::Cancelled, &mut ops);
        assert_eq!(tree.os_drag_pending, Some(second), "drag 2 still waits");
        assert!(tree.active_drag.is_some(), "with its in-app session");
        assert_eq!(cancels.get(), 1, "and its start is not cancelled");
        assert_eq!(ended.borrow().len(), 1);

        // Drag 2's own events do.
        tree.confirm_os_drag(second, &mut ops);
        assert_eq!(tree.outbound_drag_source, Some(source));
        tree.handle_os_drag_ended_for(Some(first), DropOutcome::OsCopy, &mut ops);
        assert_eq!(ended.borrow().len(), 1, "a stale end does not end drag 2");
        tree.handle_os_drag_ended_for(Some(second), DropOutcome::OsCopy, &mut ops);
        assert_eq!(
            *ended.borrow(),
            vec![DropOutcome::Cancelled, DropOutcome::OsCopy]
        );
    }

    /// X11: winit keeps its implicit grab through the XDND drag, so the
    /// window that handed its drag to the platform still gets the press's
    /// moves and release while the drag is re-entered over it. Those must not
    /// drive the re-entered session: the XDND drop does, once, and the source
    /// hears the in-app outcome.
    #[test]
    fn a_reentered_drag_ignores_the_windows_own_pointer_samples() {
        use crate::drag_payload::DropOutcome;
        let mut ops = RecordingWindowOps::new(true);
        let dropped = Rc::new(Cell::new(0_u32));
        let d = dropped.clone();
        let ended: Rc<RefCell<Vec<DropOutcome>>> = Rc::new(RefCell::new(Vec::new()));
        let e = ended.clone();
        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let row = tree.add(
            drag_arming_leaf(slot.clone(), eager_payload)
                .on_drop(move |_, _, _| {
                    d.set(d.get() + 1);
                    true
                })
                .on_drag_ended(move |o, _| e.borrow_mut().push(o)),
        );
        slot.set(Some(row));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let at = Point::new(50.0, 50.0);
        tree.dispatch_event_with_ops(
            WidgetEvent::pointer_down(at, PointerButton::Primary, Modifiers::NONE),
            &mut ops,
        );
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(60.0, 50.0)), &mut ops);
        assert_eq!(
            tree.outbound_drag_source,
            Some(row),
            "handed to the platform"
        );

        // XDND brings the drag back over this window.
        tree.begin_external_drag(
            at,
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree.os_drag_reentered);

        // winit's own samples of the grabbed press.
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(70.0, 50.0)), &mut ops);
        tree.dispatch_event_with_ops(
            WidgetEvent::pointer_up(
                Point::new(70.0, 50.0),
                PointerButton::Primary,
                Modifiers::NONE,
            ),
            &mut ops,
        );
        assert_eq!(dropped.get(), 0, "the window's release is not the drop");
        assert!(
            tree.os_drag_reentered,
            "the re-entered session is still there"
        );

        // The XDND drop completes it.
        tree.end_external_drag(
            Point::new(70.0, 50.0),
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert_eq!(dropped.get(), 1);
        tree.handle_os_drag_ended(DropOutcome::OsCopy, &mut ops);
        assert_eq!(*ended.borrow(), vec![DropOutcome::InApp { accepted: true }]);
        tree.assert_no_leaked_pointer_state();
    }

    /// The window's own samples are kept from a re-entered session for the
    /// dragging pointer only: another pointer (a pen hovering) still reaches
    /// the widgets under it.
    #[test]
    fn a_reentered_drag_leaves_other_pointers_alone() {
        let mut ops = RecordingWindowOps::new(true);
        let hovered = Rc::new(Cell::new(0_u32));
        let h = hovered.clone();
        let mut tree = WidgetTree::new();
        let slot: Rc<Cell<Option<WidgetId>>> = Rc::new(Cell::new(None));
        let row = tree.add(drag_arming_leaf(slot.clone(), eager_payload).on_hover(
            move |entered, _| {
                if entered {
                    h.set(h.get() + 1);
                }
            },
        ));
        slot.set(Some(row));
        tree.layout(SizeProposal::exact(200.0, 100.0));

        let at = Point::new(50.0, 50.0);
        tree.dispatch_event_with_ops(
            WidgetEvent::pointer_down(at, PointerButton::Primary, Modifiers::NONE),
            &mut ops,
        );
        tree.dispatch_event_with_ops(WidgetEvent::pointer_move(Point::new(60.0, 50.0)), &mut ops);
        tree.begin_external_drag(
            at,
            crate::drag_payload::ExternalDropData::default(),
            &mut ops,
        );
        assert!(tree.os_drag_reentered);
        let before = hovered.get();

        tree.pen_hover(Point::new(70.0, 50.0));
        assert!(
            hovered.get() > before,
            "the pen's hover reaches the widget under it"
        );
        assert!(tree.os_drag_reentered, "and leaves the session alone");
        tree.handle_os_drag_ended(crate::drag_payload::DropOutcome::Cancelled, &mut ops);
    }
}