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

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

Created:
4 years, 1 month ago by pdr.
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
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/+/5a3e6f1238c826cd4dd99c541d359471ffe181e8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+196 lines, -212 lines) Patch
M third_party/WebKit/Source/core/layout/TextAutosizer.cpp View 1 chunk +9 lines, -4 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, 1 month ago (2016-11-22 19:41:08 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5a3e6f1238c826cd4dd99c541d359471ffe181e8.

Powered by Google App Engine
This is Rietveld 408576698