Chromium Code Reviews
DescriptionRevert of Do not let text-size-adjust override accessibility font settings (patchset #2 id:20001 of https://codereview.chromium.org/2329733002/ )
Reason for revert:
Broke ContextualSearchManagerTest#testChainedSearchCreatesNewContent and ContextualSearchManagerTest#testChainedTapsRemovedFromHistory
BUG=646342
Original issue's description:
> Do not let text-size-adjust override accessibility font settings
>
> The text autosizer is used for both making desktop pages more legible
> on mobile, and for applying the accessibility font scale factor. When
> text-size-adjust support was added, pages were able to override the
> accessibility setting in addition to the autosizing multiplier.
>
> This patch ensures accessibility font scale settings are respected even
> with text-size-adjust. A followup bug has been filed (crbug.com/645717)
> for moving the accessibility font scale factor out of the autosizer.
>
> This patch also re-enables support for text-size-adjust which was
> disabled temporarily due to breaking accessibility settings.
>
> BUG=645269
>
> Committed: https://crrev.com/eccac1fd6db85731fb95b11915fc2197a6ceb0c3
> Cr-Commit-Position: refs/heads/master@{#418173}
TBR=skobes@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=645269
Committed: https://crrev.com/3ec856cdfb31f023a832547f4f702858e9b8a903
Cr-Commit-Position: refs/heads/master@{#418278}
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|
|||||||||||||||||||||||||||||||||||||