Rename web_usb_permission_bubble.cc/h and webusb_permission_bubble.mojom

content/app/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/chrome_build.gni")
 
 content_app_sources = [
   "android/app_jni_registrar.cc",
   "android/app_jni_registrar.h",
   "android/child_process_service.cc",
@@ skipping 39 matching lines @@
 }
 
 if (is_ios) {
   content_app_sources -= [
     "content_main.cc",
     "mojo/mojo_init.cc",
     "mojo/mojo_init.h",
   ]
 } else {
   content_app_deps += [
+    "//device/usb",

[Reilly Grant (use Gerrit), 2016/02/27 02:22:56]

This seems like an awfully high level place to add a dependency.
//content/renderer/usb already depends on //device/usb. What changed to
necessitate this?

[juncai, 2016/02/29 18:55:23]

ok, I moved this dep to the above:
else if (is_android)
since it is needed by Android.

In my machine, the Android build failed without this dep:
../../content/app/android/library_loader_hooks.cc:80: error: undefined reference
to 'device::android::RegisterUsbJni(_JNIEnv*)'

     "//mojo/shell/public/interfaces",
     "//mojo/edk/system",
     "//mojo/environment:chromium",
   ]
 }
 
 content_app_extra_configs = [
   "//build/config/compiler:wexit_time_destructors",
   "//content:content_implementation",
   "//content/public/common:mojo_shell_client",
@@ skipping 32 matching lines @@
   source_set("child") {
     visibility = [ "//content/public/app:child" ]
 
     sources = content_app_sources
     configs += content_app_extra_configs
     deps = content_app_deps
 
     defines = [ "CHROME_MULTIPLE_DLL_CHILD" ]
   }
 }