Replace internal uses of BlinkMacSystemFont on Mac with system-ui

Replace internal uses of BlinkMacSystemFont on Mac with system-ui

This patch replaces internal uses of "BlinkMacSystemFont", Blink's
non-standard extension on Mac, with the standard "system-ui" generic
font family.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679
Committed: https://crrev.com/dcc99dfb57774beee88deeb95e8f5753d6d10f8b
Cr-Commit-Position: refs/heads/master@{#426743}

[kojii, 2016-10-01 15:35:42]

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

[commit-bot: I haz the power, 2016-10-01 15:35:52]

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-10-01 15:43:22]

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

[commit-bot: I haz the power, 2016-10-01 15:43:22]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[kojii, 2016-10-12 05:23:17]

Description was changed from

==========
WIP: legacy-mac-system-ui

BUG=
==========

to

==========
Enable "system-ui" generic font family

This patch enables the "system-ui" generic font family on all
platforms.

Following changes are planned but not included in this CL:
1. Content sets the localized system font to Blink on Linux/CrOS[1].
2. Remove internal use of "BlinkMacSystemFont" and add UMA[2].
3. On Mac, expand "system-ui" to underlying name by size. This is
   expected to improve http://crbug.com/627609 .
4. On Mac, expand "system-ui" to underlying cascading list. This is
   more complex than other items and is expected to take longer.

[1] https://codereview.chromium.org/2138613002
[2] https://codereview.chromium.org/2388623002

BUG=654679
==========

[kojii, 2016-10-12 05:23:38]

Description was changed from

==========
Enable "system-ui" generic font family

This patch enables the "system-ui" generic font family on all
platforms.

Following changes are planned but not included in this CL:
1. Content sets the localized system font to Blink on Linux/CrOS[1].
2. Remove internal use of "BlinkMacSystemFont" and add UMA[2].
3. On Mac, expand "system-ui" to underlying name by size. This is
   expected to improve http://crbug.com/627609 .
4. On Mac, expand "system-ui" to underlying cascading list. This is
   more complex than other items and is expected to take longer.

[1] https://codereview.chromium.org/2138613002
[2] https://codereview.chromium.org/2388623002

BUG=654679
==========

to

==========

==========

[kojii, 2016-10-12 06:34:05]

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

[commit-bot: I haz the power, 2016-10-12 06:34:18]

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-10-12 06:42:52]

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

[commit-bot: I haz the power, 2016-10-12 06:42:52]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[kojii, 2016-10-12 08:17:34]

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

[commit-bot: I haz the power, 2016-10-12 08:17:47]

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-10-12 08:21:07]

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

[commit-bot: I haz the power, 2016-10-12 08:21:08]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[kojii, 2016-10-12 08:44:05]

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

[commit-bot: I haz the power, 2016-10-12 08:44:24]

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-10-12 08:54:38]

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

[commit-bot: I haz the power, 2016-10-12 08:54:39]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[kojii, 2016-10-19 05:09:43]

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

[commit-bot: I haz the power, 2016-10-19 05:12:42]

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

[kojii, 2016-10-19 06:14:54]

Description was changed from

==========

==========

to

==========
Eliminate internal uses of BlinkMacSystemFont on Mac

This patch eliminates internal uses of BlinkMacSystemFont, Blink's
non-standard extension on Mac.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679
==========

[kojii, 2016-10-19 06:22:42]

Description was changed from

==========
Eliminate internal uses of BlinkMacSystemFont on Mac

This patch eliminates internal uses of BlinkMacSystemFont, Blink's
non-standard extension on Mac.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679
==========

to

==========
Replace internal uses of BlinkMacSystemFont on Mac with system-ui

This patch replaces internal uses of "BlinkMacSystemFont", Blink's
non-standard extension on Mac, with the standard "system-ui" generic
font family.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679
==========

[kojii, 2016-10-19 06:24:55]

kojii@chromium.org changed reviewers:
+ drott@chromium.org, eae@chromium.org, holte@chromium.org

[kojii, 2016-10-19 06:24:56]

PTAL.

drott@: platform
eae@: core
holte@: tools/metrics

Bots is in trouble, but I believe it passes, only minor change from PS4.

drott@, I hope the description answers to your question in the previous CL, if
not, happy to discuss more.

[kojii, 2016-10-19 06:58:47]

The CQ bit was unchecked by kojii@chromium.org

[kojii, 2016-10-19 06:58:51]

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

[kojii, 2016-10-19 07:50:14]

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

[commit-bot: I haz the power, 2016-10-19 09:30:27]

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-10-19 12:35:12]

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

[commit-bot: I haz the power, 2016-10-19 12:35:13]

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

[eae, 2016-10-19 15:47:37]

OK, LGTM

[Steven Holte, 2016-10-19 18:57:31]

histograms lgtm

[drott, 2016-10-20 15:09:12]

LGTM

[kojii, 2016-10-21 02:19:07]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2016-10-21 02:19:44]

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-10-21 02:23:14]

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

[commit-bot: I haz the power, 2016-10-21 02:23:15]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[kojii, 2016-10-21 03:11:07]

The CQ bit was checked by kojii@chromium.org

[kojii, 2016-10-21 03:11:08]

The patchset sent to the CQ was uploaded after l-g-t-m from drott@chromium.org,
eae@chromium.org, holte@chromium.org
Link to the patchset: https://codereview.chromium.org/2388623002/#ps120001
(title: "Rebase")

[commit-bot: I haz the power, 2016-10-21 03:11:28]

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-10-21 03:27:44]

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

[commit-bot: I haz the power, 2016-10-21 03:27:46]

Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[kojii, 2016-10-21 03:39:08]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2016-10-21 03:39:27]

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-10-21 05:16:42]

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

[commit-bot: I haz the power, 2016-10-21 05:16:44]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[kojii, 2016-10-21 05:39:52]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2016-10-21 05:40:06]

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-10-21 07:12:00]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-10-21 13:27:55]

Description was changed from

==========
Replace internal uses of BlinkMacSystemFont on Mac with system-ui

This patch replaces internal uses of "BlinkMacSystemFont", Blink's
non-standard extension on Mac, with the standard "system-ui" generic
font family.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679
==========

to

==========
Replace internal uses of BlinkMacSystemFont on Mac with system-ui

This patch replaces internal uses of "BlinkMacSystemFont", Blink's
non-standard extension on Mac, with the standard "system-ui" generic
font family.

By eliminating internal uses, the aliasing is moved from FontCache
to CSS resolving time. This allows access to the document, which is
needed for per-document use counter.

BUG=554590, 654679

Committed: https://crrev.com/dcc99dfb57774beee88deeb95e8f5753d6d10f8b
Cr-Commit-Position: refs/heads/master@{#426743}
==========

[commit-bot: I haz the power, 2016-10-21 13:27:56]

Patchset 7 (id:??) landed as
https://crrev.com/dcc99dfb57774beee88deeb95e8f5753d6d10f8b
Cr-Commit-Position: refs/heads/master@{#426743}