No subject

    offset.move(view.scrollOffset());

There=E2=80=99s no need to convert the IntSize to an IntPoint since IntPoin=
t::move already takes an IntSize. That=E2=80=99s separate from the question=
 you are discussing with Simon about whether this is OK.

I fact, again ignoring the correctness of the coordinate math that you are =
discussing with Simon, I think this 6-line snippet can be written like in t=
wo lines like this:

    FrameView& view =3D *range->ownerDocument().view();
    rect.setLocation(view.contentsToRootView(rect.location() + view.scrollO=
ffset()));

--=20
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=3Demail
------- You are receiving this mail because: -------
You are the assignee for the bug.=