Use new wrapper types for web_bluetooth.mojom

content/common/bluetooth/web_bluetooth_device_id.cc

 // 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.
 
 #include "content/common/bluetooth/web_bluetooth_device_id.h"
 
 #include "base/base64.h"
 #include "base/strings/string_util.h"
 #include "crypto/random.h"
 
@@ skipping 61 matching lines @@
 bool WebBluetoothDeviceId::operator==(
     const WebBluetoothDeviceId& device_id) const {
   return str() == device_id.str();
 }
 
 bool WebBluetoothDeviceId::operator!=(
     const WebBluetoothDeviceId& device_id) const {
   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