[Webkit-unassigned] [Bug 107831] [chromium] Make selection handling work in applyPageScaleInCompositor mode

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 30 16:45:29 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=107831





--- Comment #5 from Chris Hopman <cjhopman at chromium.org>  2013-01-30 16:47:29 PST ---
(From update of attachment 184508)
View in context: https://bugs.webkit.org/attachment.cgi?id=184508&action=review

>> Source/WebKit/chromium/src/WebFrameImpl.cpp:1386
>> +    VisiblePosition extentPosition = visiblePositionForWindowPoint(unscaledExtent);
> 
> There got to be a better way of achieving this. At minimum, we should have some helper function that automatically scale both x and y.

Changed this to convert to IntPoint earlier and can then use IntPoint::scale. I was surprised that WebPoint had no way to scale, but I like this better anyway since I now don't need the conversion when calling windowToContents.

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



More information about the webkit-unassigned mailing list