Similar to bug 107592, auto-zoom for focused node need to consider page scale and device scale differently.
Created attachment 184082 [details] Patch
Comment on attachment 184082 [details] Patch Tests?
Comment on attachment 184082 [details] Patch The code change itself looks fine.
Didn't include a test because the original test for the function was disabled (bug 98558). Now I'd like to fix that bug and add a test for the changed code.
Created attachment 185280 [details] Patch
LGTM. Adam?
Comment on attachment 185280 [details] Patch rs=me based on aelias's comments.
Comment on attachment 185280 [details] Patch Clearing flags on attachment: 185280 Committed r141153: <http://trac.webkit.org/changeset/141153>
All reviewed patches have been landed. Closing bug.
Re-opened since this is blocked by bug 108280
(In reply to comment #10) > Re-opened since this is blocked by bug 108280 WebFrameTest.DivScrollIntoEditableTest is failing on Chromium Mac bots. WebFrameTest.DivScrollIntoEditableTest: ../../third_party/WebKit/Source/WebKit/chromium/tests/WebFrameTest.cpp:972: Failure The difference between hScroll and scroll.x() is 23, which exceeds 1, where hScroll evaluates to 102, scroll.x() evaluates to 125, and 1 evaluates to 1. ../../third_party/WebKit/Source/WebKit/chromium/tests/WebFrameTest.cpp:974: Failure The difference between vScroll and scroll.y() is 3, which exceeds 1, where vScroll evaluates to 105, scroll.y() evaluates to 102, and 1 evaluates to 1.
Created attachment 185405 [details] Added font-size to ensure same caret size across platforms
Comment on attachment 185405 [details] Added font-size to ensure same caret size across platforms Clearing flags on attachment: 185405 Committed r141231: <http://trac.webkit.org/changeset/141231>