Import Mojo Changes

Make |use_glib| configurable.

This is needed for mojo. We're trying to remove glib and dbus from our build, and need to be able to configure this option. This will be rolled back into the mojo repository.

This was split from the mojo patch: https://codereview.chromium.org/853133002/

BUG=443439
Committed: https://crrev.com/9252aff86f89f9ae0293a81b46097b1b51a6b553
Cr-Commit-Position: refs/heads/master@{#312261}

[Elliot Glaysher, 2015-01-16 01:04:51]

erg@chromium.org changed reviewers:
+ thakis@chromium.org

[Elliot Glaysher, 2015-01-20 18:04:34]

erg@chromium.org changed reviewers:
+ scottmg@chromium.org

[Elliot Glaysher, 2015-01-20 18:04:34]

Nico appears to be unavailable.

scottmg: build/
thestig: base/

[Elliot Glaysher, 2015-01-20 18:04:42]

erg@chromium.org changed reviewers:
+ thestig@chromium.org

[Elliot Glaysher, 2015-01-20 18:04:42]

Nico appears to be unavailable.

scottmg: build/
thestig: base/

[scottmg, 2015-01-20 18:09:52]

The cl description got truncated.

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni
File build/config/ui.gni (left):

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni#oldcode57
build/config/ui.gni:57: use_glib = is_linux && !use_ozone
could it be "&& some_condition" down here instead of making it yet-another arg
to be configured?

[Elliot Glaysher, 2015-01-20 18:14:14]

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni
File build/config/ui.gni (left):

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni#oldcode57
build/config/ui.gni:57: use_glib = is_linux && !use_ozone
On 2015/01/20 18:09:51, scottmg wrote:
> could it be "&& some_condition" down here instead of making it yet-another arg
> to be configured?

I don't know how I'd do that. This file is copied directly into the mojo
repository (to support a bunch of copied chrome code), and there doesn't appear
to be a we_are_building_chrome_code_out_of_the_chrome_repo flag.

[scottmg, 2015-01-20 18:16:36]

lgtm

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni
File build/config/ui.gni (left):

https://codereview.chromium.org/853103004/diff/1/build/config/ui.gni#oldcode57
build/config/ui.gni:57: use_glib = is_linux && !use_ozone
On 2015/01/20 18:14:13, Elliot Glaysher wrote:
> On 2015/01/20 18:09:51, scottmg wrote:
> > could it be "&& some_condition" down here instead of making it yet-another
arg
> > to be configured?
> 
> I don't know how I'd do that. This file is copied directly into the mojo
> repository (to support a bunch of copied chrome code), and there doesn't
appear
> to be a we_are_building_chrome_code_out_of_the_chrome_repo flag.

OK. I meant more checking whatever it is that's not using glib now. But I guess
it's just "new UI that doesn't need it any more"?

[Lei Zhang, 2015-01-20 20:12:19]

lgtm

[Elliot Glaysher, 2015-01-20 20:17:35]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2015-01-20 20:18:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/853103004/1

[commit-bot: I haz the power, 2015-01-20 20:25:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-20 20:27:21]

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