Summary: | Text Autosizing: Counteract funky window sizing on Android. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | John Mellor <johnme> | ||||||
Component: | New Bugs | Assignee: | John Mellor <johnme> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, aelias, benjamin, eric, skyostil, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 84186 | ||||||||
Attachments: |
|
Description
John Mellor
2012-10-09 11:44:27 PDT
Created attachment 167806 [details]
Patch
Comment on attachment 167806 [details]
Patch
Thanks.
I would have preferred pushing down a setting so that we can switch between the modes with only a Chromium-side change; this will be a loose end. That said if this is one of the last things blocking the text autosizing completion we can land and fix later. (In reply to comment #3) > I would have preferred pushing down a setting so that we can switch between the modes with only a Chromium-side change; this will be a loose end. That said if this is one of the last things blocking the text autosizing completion we can land and fix later. Yeah, this is the last thing blocking us from turning Text Autosizing on in Chrome for Android. By pushing down a setting, do you mean exposing WebSettingsImpl::applyDefaultDeviceScaleFactorInCompositor to WebCore? Or some separate setting? Yes, WebSettingsImpl::applyDefaultDeviceScaleFactorInCompositor is essentially equivalent to this new USE define, so we would be better to use that instead. Comment on attachment 167806 [details] Patch Rejecting attachment 167806 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: commit-queue/Source/WebKit/chromium/third_party/v8-i18n --revision 150 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' 50>At revision 150. ________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' ________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' Updating webkit projects from gyp files... Full output: http://queues.webkit.org/results/14197033 Comment on attachment 167806 [details]
Patch
This code already has a m_document, so it shouldn't have any trouble finding Settings. Perhaps we should use that instead.
Created attachment 167876 [details]
Patch
Uploaded a patch using applyPageScaleFactorInCompositor. It's essentially the same thing as the other setting except for being in WebCore and I plan to flip them simultaneously. Comment on attachment 167876 [details]
Patch
Ok. I would have added a comment explaining the connection between applyPageScaleFactorInCompositor and the bug in the units of the window size.
Comment on attachment 167876 [details]
Patch
I'll cq+ without the comment to unblock johnme@ for his remaining cleanup. I'll go back eventually to remove all these if() statements anyway.
Comment on attachment 167876 [details] Patch Clearing flags on attachment: 167876 Committed r130866: <http://trac.webkit.org/changeset/130866> All reviewed patches have been landed. Closing bug. |