Add DirectWrite flag and plumb it through to Blink

Add DirectWrite flag and plumb it through to Blink

Add an "enable-direct-write" flag and plumb it through to
WebRuntimeFeatures on the Blink side.

R=abarth@chromium.org
BUG=25541
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=239354

[eae, 2013-11-13 22:05:37]

The blink side of this change has landed so the bots should like it better now.

[dglazkov, 2013-11-14 21:05:37]

lgtm!

[commit-bot: I haz the power, 2013-11-14 21:06:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/120001

[commit-bot: I haz the power, 2013-11-14 21:26:40]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[eae, 2013-11-14 21:34:22]

Needs OWNERS approval:

Missing LGTM from an OWNER for these files:
    content/browser/renderer_host/render_process_host_impl.cc
    content/child/runtime_features.cc
    content/public/common/content_switches.cc
    content/public/common/content_switches.h

[commit-bot: I haz the power, 2013-11-14 21:34:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/120001

[eae, 2013-11-14 21:45:26]

Need OWNERS approval

[commit-bot: I haz the power, 2013-11-14 22:05:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/120001

[commit-bot: I haz the power, 2013-11-15 01:49:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/120001

[commit-bot: I haz the power, 2013-11-15 04:03:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/120001

[commit-bot: I haz the power, 2013-11-15 05:33:40]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[Avi (use Gerrit), 2013-11-16 06:26:50]

https://codereview.chromium.org/66333016/diff/120001/chrome/app/generated_res...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/66333016/diff/120001/chrome/app/generated_res...
chrome/app/generated_resources.grd:5860: +        Enables the use of
experimental DirectWrite font rendering system on windows.
Don't mention the platform here, do an actual OS restriction.

https://codereview.chromium.org/66333016/diff/120001/chrome/browser/about_fla...
File chrome/browser/about_flags.cc (right):

https://codereview.chromium.org/66333016/diff/120001/chrome/browser/about_fla...
chrome/browser/about_flags.cc:426: kOsAll,
kOsWin

https://codereview.chromium.org/66333016/diff/120001/content/child/runtime_fe...
File content/child/runtime_features.cc (right):

https://codereview.chromium.org/66333016/diff/120001/content/child/runtime_fe...
content/child/runtime_features.cc:134: if
(command_line.HasSwitch(switches::kEnableDirectWrite))
Can you #if defined(OS_WIN)?

https://codereview.chromium.org/66333016/diff/120001/content/public/common/co...
File content/public/common/content_switches.cc (right):

https://codereview.chromium.org/66333016/diff/120001/content/public/common/co...
content/public/common/content_switches.cc:402: // Enables the DirectWrite font
rendering system on windows.
#if defined(OS_WIN)

https://codereview.chromium.org/66333016/diff/120001/content/public/common/co...
File content/public/common/content_switches.h (right):

https://codereview.chromium.org/66333016/diff/120001/content/public/common/co...
content/public/common/content_switches.h:125: CONTENT_EXPORT extern const char
kEnableDirectWrite[];
#if defined(OS_WIN)

[eae, 2013-12-05 23:25:59]

Thanks Avi, I made the changes you suggested. Please take another look when you
get a chance.

[Avi (use Gerrit), 2013-12-05 23:43:04]

LGTM!

[eae, 2013-12-05 23:43:31]

On 2013/12/05 23:43:04, Avi wrote:
> LGTM!

Thanks!

[commit-bot: I haz the power, 2013-12-06 00:03:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/580001

[commit-bot: I haz the power, 2013-12-06 05:10:40]

Retried try job too often on android_aosp for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_ao...

[commit-bot: I haz the power, 2013-12-07 01:02:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/580001

[commit-bot: I haz the power, 2013-12-07 15:07:55]

The commit queue went berserk retrying too often for a
seemingly flaky test on builder android_dbg:
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2013-12-07 22:30:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/66333016/580001

[commit-bot: I haz the power, 2013-12-07 22:32:09]

Change committed as 239354