Use new wrapper types for web_bluetooth.mojom

content/common/bluetooth/web_bluetooth_device_id.cc

Index: content/common/bluetooth/web_bluetooth_device_id.cc
diff --git a/content/common/bluetooth/web_bluetooth_device_id.cc b/content/common/bluetooth/web_bluetooth_device_id.cc
index 823ed0fd979cd6736a587d525848457153e084b2..94749e75f831cc0391332971279540fdecde15a3 100644
--- a/content/common/bluetooth/web_bluetooth_device_id.cc
+++ b/content/common/bluetooth/web_bluetooth_device_id.cc
@@ -79,9 +79,9 @@ bool WebBluetoothDeviceId::operator!=(
   return !(*this == device_id);
 }
 
+}  // namespace content
+
 std::ostream& operator<<(std::ostream& out,
-                         const WebBluetoothDeviceId& device_id) {
+                         const content::WebBluetoothDeviceId& device_id) {

[dcheng, 2016/11/24 04:56:49]

Strictly speaking, shouldn't this be in namespace content to ensure ADL applies?

[juncai, 2016/11/28 22:25:21]

If it is in the namespace content as before, the following will generate compile
error since it can't find the operator<< overload for WebURL:
https://cs.chromium.org/chromium/src/content/renderer/presentation/presentati...
https://cs.chromium.org/chromium/src/content/renderer/presentation/presentati...

I guess since the above code is in the namespace content, it finds the
operator<< overload for WebBluetoothDeviceId, not WebURL, so it generates
compile error. By moving this operator overloading out of the namespace content,
it will search the global namespace.

[dcheng, 2016/11/28 22:29:33]

I don't think I understand what this has to do with operator<< for GURL (there
is no operator<< for WebURL).

[juncai, 2016/11/29 04:23:28]

Done.

   return out << device_id.str();
 }
-
-}  // namespace content