GYP fix: toolkit_uses_gtk% should be dependent on use_aura%.

GYP fix: toolkit_uses_gtk% should be dependent on use_aura%.

This fixes the linux_aura FYI builder. This was broken in r127620.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=127858

[Emmanuel Saint-loubert-Bié, 2012-03-20 22:25:14]

lgtm

https://chromiumcodereview.appspot.com/9789002/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/9789002/diff/1/build/common.gypi#newco...
build/common.gypi:328: ['OS=="linux" and use_aura==1', {
If you want to go this way the test could simply be use_aura==1 (remove linux).
Since toolkit_uses_gtk and use_aura are at this time mutually exclusive.

https://chromiumcodereview.appspot.com/9789002/diff/1/build/common.gypi#newco...
build/common.gypi:331: 'toolkit_uses_gtk%': 1,
I am wondering if we should have this set to 1 for platforms that clearly do not
need it (e.g Win. Mac...)

[Elliot Glaysher, 2012-03-20 22:38:32]

That's a good point. I reversed the polarity.

[commit-bot: I haz the power, 2012-03-20 22:39:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/erg@chromium.org/9789002/5001

[commit-bot: I haz the power, 2012-03-20 23:59:46]

Try job failure for 9789002-5001 (retry) (retry) on linux_rel for step
"check_deps".
It's a second try, previously, step "check_deps" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-03-21 00:01:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/erg@chromium.org/9789002/5001

[commit-bot: I haz the power, 2012-03-21 01:13:21]

Change committed as 127858