Remove some global defines and build flags.

Remove some global defines and build flags.

Convert image loader extension C++ checks to use OS_CHROMEOS. There are
only two of these and this flag always goes with ChromeOS and is not
overridable. It seems we do not get much benefit from having this be a
define/buildflag, and because this is referenced from both //ui and
//chrome the layering of where the flag should be but is unclear.

Move use_minikin_hyphenation to a buildflag header in Blink.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2cf0a69b66582c80125579d3fa32885ee85917b3
Cr-Commit-Position: refs/heads/master@{#431680}

[brettw, 2016-11-11 19:12:02]

Description was changed from

==========
Remove some global defines and build flags.

Convert image loader extension C++ checks to use OS_CHROMEOS. There are
only two of these and this flag always goes with ChromeOS and is not
overridable. It seems we do not get much benefit from having this be a
define/buildflag, and because this is referenced from both //ui and
//chrome the layering of where the flag should be but is unclear.

Move use_minikin_hyphenation to a buildflag header in Blink.
==========

to

==========
Remove some global defines and build flags.

Convert image loader extension C++ checks to use OS_CHROMEOS. There are
only two of these and this flag always goes with ChromeOS and is not
overridable. It seems we do not get much benefit from having this be a
define/buildflag, and because this is referenced from both //ui and
//chrome the layering of where the flag should be but is unclear.

Move use_minikin_hyphenation to a buildflag header in Blink.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[brettw, 2016-11-11 19:13:50]

brettw@chromium.org changed reviewers:
+ sky@chromium.org

[brettw, 2016-11-11 19:13:55]

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

[commit-bot: I haz the power, 2016-11-11 19:14:26]

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

[sky, 2016-11-11 19:21:39]

LGTM

[brettw, 2016-11-11 20:17:27]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2016-11-11 20:17:36]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-11-11 20:17:47]

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-11 23:14:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-11 23:54:25]

Description was changed from

==========
Remove some global defines and build flags.

Convert image loader extension C++ checks to use OS_CHROMEOS. There are
only two of these and this flag always goes with ChromeOS and is not
overridable. It seems we do not get much benefit from having this be a
define/buildflag, and because this is referenced from both //ui and
//chrome the layering of where the flag should be but is unclear.

Move use_minikin_hyphenation to a buildflag header in Blink.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove some global defines and build flags.

Convert image loader extension C++ checks to use OS_CHROMEOS. There are
only two of these and this flag always goes with ChromeOS and is not
overridable. It seems we do not get much benefit from having this be a
define/buildflag, and because this is referenced from both //ui and
//chrome the layering of where the flag should be but is unclear.

Move use_minikin_hyphenation to a buildflag header in Blink.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2cf0a69b66582c80125579d3fa32885ee85917b3
Cr-Commit-Position: refs/heads/master@{#431680}
==========

[commit-bot: I haz the power, 2016-11-11 23:54:26]

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