Aura on Android: Remove unnecessary dependencies on ui/android.

components/web_contents_delegate_android/BUILD.gn

Index: components/web_contents_delegate_android/BUILD.gn
diff --git a/components/web_contents_delegate_android/BUILD.gn b/components/web_contents_delegate_android/BUILD.gn
index 56ee8de191b4ef13dff2011dd775c35715318a73..c3d6c2f29d151a605c9a8633159a6c8c09caa3b3 100644
--- a/components/web_contents_delegate_android/BUILD.gn
+++ b/components/web_contents_delegate_android/BUILD.gn
@@ -24,11 +24,14 @@ static_library("web_contents_delegate_android") {
     "//content/public/common",
     "//net",
     "//skia",
-    "//ui/android",
     "//ui/base",
     "//ui/gfx",
     "//ui/gfx/geometry",
   ]
+
+  if (!use_aura) {
+    deps += [ "//ui/android" ]

[no sievers, 2015/11/19 22:23:09]

this should not be needed since the aura build should not depend on
web_contents_delegate_android

[Hadi, 2015/11/19 22:38:04]

(I replied in the discussion thread in the bottom which didn't appear in the
inlined comments, so I am repeating it here).

Yes, I think you're right. We're depending on web_contents_delegate_android from
different places, so I thought I can do this as a temporary quick fix. If I
comment out the web_contents_delegate_android, I get the following errors:

//chrome/browser:browser(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android(//build/toolchain/android:arm)
//android_webview:common(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android(//build/toolchain/android:arm)
//chrome/android:chrome_shared_test_java__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java(//build/toolchain/android:arm)
//android_webview/glue:glue_resource_rewriter__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java_resources(//build/toolchain/android:arm)
//android_webview/native:native(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android(//build/toolchain/android:arm)
//android_webview:android_webview_java__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java(//build/toolchain/android:arm)
//android_webview/glue:glue__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java_resources(//build/toolchain/android:arm)
//chrome/android:chrome_java__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java(//build/toolchain/android:arm)
//chrome/test/android:chrome_java_test_support__build_config(//build/toolchain/android:arm)
  needs
//components/web_contents_delegate_android:web_contents_delegate_android_java(//build/toolchain/android:arm)

Probably we don't need some of these targets too, and I think removing each of
them will require some other changes so it will be a bigger project. Maybe I can
tackle fixing the dependency tree and removing dependency on targets we don't
need (including ui_android) as one CL if it looks high priority.

+  }
 }
 
 # GYP: //components/web_contents_delegate_android:web_contents_delegate_android_java