Only apply accessibility font scale factor when autosizing inflates

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
Committed: https://crrev.com/394de7b0baa6b0136b6acfd15486de6a98e19152
Cr-Commit-Position: refs/heads/master@{#433757}

[pdr., 2016-11-19 04:43:48]

pdr@chromium.org changed reviewers:
+ aelias@chromium.org, skobes@chromium.org

[pdr., 2016-11-19 04:43:55]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-19 04:44:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-19 06:35:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-19 06:35:07]

Dry run: This issue passed the CQ dry run.

[skobes, 2016-11-22 00:04:55]

lgtm

[pdr., 2016-11-22 00:30:18]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-11-22 00:30:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 01:08:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-22 01:08:31]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[pdr., 2016-11-22 01:17:53]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-11-22 01:18:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 02:54:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479777473012420, "parent_rev":
"8b8655e1166f86ee28c3b69d68183900d0e9c5b9", "commit_rev":
"413cb0c8e35da3a8acdddeea31f81f4cae7c184c"}

[commit-bot: I haz the power, 2016-11-22 02:55:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-22 02:59:18]

Description was changed from

==========
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
==========

to

==========
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

Committed: https://crrev.com/394de7b0baa6b0136b6acfd15486de6a98e19152
Cr-Commit-Position: refs/heads/master@{#433757}
==========

[commit-bot: I haz the power, 2016-11-22 02:59:19]

Patchset 1 (id:??) landed as
https://crrev.com/394de7b0baa6b0136b6acfd15486de6a98e19152
Cr-Commit-Position: refs/heads/master@{#433757}