bluetooth: Add device list retrieval for chrome://bluetooth-internals

device/bluetooth/public/interfaces/BUILD.gn

 # Copyright 2016 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("//mojo/public/tools/bindings/mojom.gni")
 
-mojom("bluetooth_mojom_bluetooth_uuid") {
+mojom("interfaces") {
   sources = [
     "bluetooth_uuid.mojom",
   ]
 
   use_new_wrapper_types = false
 }
+
+mojom("experimental_interfaces") {

[scheib, 2016/09/28 03:12:21]

Did you have to drop the visibility because of adding the interface in
chrome_content_browser_client.cc? That's unfortunate. I'm not sure it's worth
going back -- because it seems cleaner that way.

ortuno, any preference?

[ortuno, 2016/09/28 09:44:55]

Yeah. Having visibility would be nice. Do you get an error if you expose it only
to //device/bluetooth:mojo and //chrome/browser/ui:ui?

[mbrunson, 2016/09/28 21:20:35]

I had visibility in at one point but it probably got dropped when I pulled some
changes from another branch. 

I should be able to get it to build with the chrome content browser client
change if I specifically target the //chrome/browser:browser target. I shouldn't
have to put it in //chrome/browser/ui because there's no Mojo code in there.

[scheib, 2016/09/28 23:26:40]

Nice to have it back, sort of, but having chrome/browser:browser also sort of
defeats the point a bit... it's pretty broad access. Better than nothing,
though.

[ortuno, 2016/09/29 01:43:06]

True, there is no mojo code in there, but 
chrome/browser/ui/webui/bluetooth_internals/bluetooth_internals_ui.cc still
needs the bindings generated.

I don't think we want chrome/browser:browser to have direct access to the
bindings. It already depends on mojo.

[mbrunson, 2016/09/29 02:50:23]

I've restricted the visibility to bluetooth_internals and device/bluetooth:mojo
and that seems to compile. I guess it doesn't matter if
//chrome/browser/BUILD.gn depends on device/bluetooth:mojo which depends on the
interfaces. As long as it doesn't depend directly on it, it's fine I guess??

+  sources = [
+    "adapter.mojom",
+  ]
+}