Move most of FontRenderStyle manipulation code into its own file

Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650
Committed: https://crrev.com/fe669432f4eca6a0948012858b459fd2631a72da
Cr-Commit-Position: refs/heads/master@{#392494}

[tzik, 2016-05-04 09:08:03]

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

[commit-bot: I haz the power, 2016-05-04 09:08:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/1

[tzik, 2016-05-04 09:09:55]

Description was changed from

==========
Move most of FontRenderStyle manipulation into its own file

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

to

==========
Move most of FontRenderStyle manipulation into its own file

BUG=608650
==========

[commit-bot: I haz the power, 2016-05-04 09:27:47]

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

[commit-bot: I haz the power, 2016-05-04 09:27:47]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[tzik, 2016-05-04 09:42:55]

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

[commit-bot: I haz the power, 2016-05-04 09:43:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/20001

[commit-bot: I haz the power, 2016-05-04 10:31:41]

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

[commit-bot: I haz the power, 2016-05-04 10:31:42]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[tzik, 2016-05-04 11:02:11]

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

[commit-bot: I haz the power, 2016-05-04 11:02:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/40001

[commit-bot: I haz the power, 2016-05-04 11:52:53]

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

[commit-bot: I haz the power, 2016-05-04 11:52:54]

Dry run: Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[tzik, 2016-05-09 07:53:15]

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

[commit-bot: I haz the power, 2016-05-09 07:53:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/60001

[commit-bot: I haz the power, 2016-05-09 07:56:58]

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

[commit-bot: I haz the power, 2016-05-09 07:56:59]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tzik, 2016-05-09 08:59:53]

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

[commit-bot: I haz the power, 2016-05-09 08:59:59]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/60001

[commit-bot: I haz the power, 2016-05-09 09:58:56]

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

[commit-bot: I haz the power, 2016-05-09 09:58:57]

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

[tzik, 2016-05-09 13:59:38]

Description was changed from

==========
Move most of FontRenderStyle manipulation into its own file

BUG=608650
==========

to

==========
Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650
==========

[tzik, 2016-05-09 13:59:38]

tzik@chromium.org changed reviewers:
+ eae@chromium.org

[tzik, 2016-05-09 14:12:56]

PTAL

[eae, 2016-05-09 15:35:56]

LGTM

[eae, 2016-05-09 15:36:00]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-05-09 15:36:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/60001

[commit-bot: I haz the power, 2016-05-09 15:43:13]

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

[commit-bot: I haz the power, 2016-05-09 15:43:14]

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

[tzik, 2016-05-09 23:29:20]

tzik@chromium.org changed reviewers:
+ haraken@chromium.org

[tzik, 2016-05-09 23:29:21]

Haraken-san, could you PTAL to Source/web part as an owner?

[haraken, 2016-05-09 23:33:31]

On 2016/05/09 23:29:21, tzik wrote:
> Haraken-san, could you PTAL to Source/web part as an owner?

web/ LGTM

[tzik, 2016-05-10 00:34:26]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2016-05-10 00:35:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944993003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944993003/60001

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

Description was changed from

==========
Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650
==========

to

==========
Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650
==========

[commit-bot: I haz the power, 2016-05-10 00:40:11]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-10 00:42:39]

Description was changed from

==========
Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650
==========

to

==========
Move most of FontRenderStyle manipulation code into its own file

This CL isolates FontPlatformData::m_style handling into its
method, so that m_style is easy to move from FontPlatformData
to SimpleFontData.

BUG=608650

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

[commit-bot: I haz the power, 2016-05-10 00:42:40]

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