Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(309)

Issue 2544633002: Only apply accessibility font scale factor when autosizing inflates (Closed)

Created:
4 years ago by pdr.
Modified:
4 years ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Only apply accessibility font scale factor when autosizing inflates The accessibility font scale factor is currently applied to all text that the text autosizer would inflate (this is changing in crbug.com/645717). crrev.com/1229b1414d64c314666d2fb3bf47706f2b5e947b introduced a bug where text that would not be inflated would become inflated if text-size-adjust: none was applied. This patch only applies the accessibility font scale factor if the text autosizer would have inflated the text already. A new test, AccessibilityFontScaleFactorWithTextSizeAdjustNone, has been added. Other tests in TestAutosizerTest.cpp have been reformatted to 80cols. BUG=666378 Review-Url: https://codereview.chromium.org/2514733005 Cr-Commit-Position: refs/heads/master@{#433757} (cherry picked from commit 394de7b0baa6b0136b6acfd15486de6a98e19152) Committed: https://chromium.googlesource.com/chromium/src/+/e185926ca8f2014314df6b7a8c345607323d4580

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+194 lines, -211 lines) Patch
M third_party/WebKit/Source/core/layout/TextAutosizer.cpp View 1 chunk +7 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/TextAutosizerTest.cpp View 13 chunks +187 lines, -208 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
pdr.
4 years ago (2016-11-30 21:23:00 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
e185926ca8f2014314df6b7a8c345607323d4580.

Powered by Google App Engine
This is Rietveld 408576698