Don't compile events.gyp:events on Android.

Don't compile events.gyp:events on Android.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=265631

[tdresser, 2014-04-17 19:40:05]

sky@, can you take a look?

It feels like I may be fixing a symptom of a larger issue here - let me know if
this is the wrong approach.

[jdduke (slow), 2014-04-17 20:03:42]

So it looks like Android really doesn't use ime.gyp, but it does get linked in
for tests and it's probably worth a separate patch.  Your first patchset is
probably sufficient for removing the events.gyp:events dependency.

https://codereview.chromium.org/241213002/diff/1/ui/base/ime/ime.gypi
File ui/base/ime/ime.gypi (right):

https://codereview.chromium.org/241213002/diff/1/ui/base/ime/ime.gypi#newcode118
ui/base/ime/ime.gypi:118: 'dependencies!' : [
Sorry, I think the IME code gets stripped out anyway, so this should be fine.

[sky, 2014-04-17 22:00:19]

https://codereview.chromium.org/241213002/diff/10001/chrome/chrome_browser_ui...
File chrome/chrome_browser_ui_views.gyp (right):

https://codereview.chromium.org/241213002/diff/10001/chrome/chrome_browser_ui...
chrome/chrome_browser_ui_views.gyp:11: 'target_name': 'browser_ui_views',
Why is the change needed here? android shouldn't depend upon browser_ui_views at
all.

[tdresser, 2014-04-22 20:44:23]

Android was depending on browser_ui_views.

How does this look?

[sky, 2014-04-22 22:15:39]

LGTM

[tdresser, 2014-04-23 12:24:09]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2014-04-23 12:24:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tdresser@chromium.org/241213002/40001

[commit-bot: I haz the power, 2014-04-23 13:46:15]

Change committed as 265631