Use new wrapper types for web_bluetooth.mojom

content/renderer/bluetooth/bluetooth_type_converters.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/renderer/bluetooth/bluetooth_type_converters.h"
 
 #include "content/child/mojo/type_converters.h"
 #include "third_party/WebKit/public/platform/modules/bluetooth/WebRequestDeviceOptions.h"
 
 namespace mojo {
 
 // static
 blink::mojom::WebBluetoothScanFilterPtr TypeConverter<
     blink::mojom::WebBluetoothScanFilterPtr,
     blink::WebBluetoothScanFilter>::Convert(const blink::WebBluetoothScanFilter&
                                                 web_filter) {
   blink::mojom::WebBluetoothScanFilterPtr filter =
       blink::mojom::WebBluetoothScanFilter::New();
 
-  if (!web_filter.services.isEmpty())
-    filter->services =
-        Array<base::Optional<device::BluetoothUUID>>::From(web_filter.services);
+  if (!web_filter.services.isEmpty()) {
+    filter->services = std::vector<device::BluetoothUUID>();
+    for (const auto service : web_filter.services) {

[ortuno, 2016/11/21 04:42:31]

Make it a reference.

[juncai, 2016/11/21 21:27:05]

Done.

+      filter->services->push_back(device::BluetoothUUID(service.utf8()));
+    }
+  }
 
   if (web_filter.hasName)
-    filter->name = String::From(web_filter.name);
+    filter->name = web_filter.name.utf8();
 
   if (!web_filter.namePrefix.isEmpty())
-    filter->name_prefix = String::From(web_filter.namePrefix);
+    filter->name_prefix = web_filter.namePrefix.utf8();
   return filter;
 }
 
 // static
 blink::mojom::WebBluetoothRequestDeviceOptionsPtr
 TypeConverter<blink::mojom::WebBluetoothRequestDeviceOptionsPtr,
               blink::WebRequestDeviceOptions>::
     Convert(const blink::WebRequestDeviceOptions& web_options) {
   blink::mojom::WebBluetoothRequestDeviceOptionsPtr options =
       blink::mojom::WebBluetoothRequestDeviceOptions::New();
 
-  options->filters = mojo::Array<blink::mojom::WebBluetoothScanFilterPtr>::From(
-      web_options.filters);
-  options->optional_services =
-      mojo::Array<base::Optional<device::BluetoothUUID>>::From(
-          web_options.optionalServices);
+  for (const auto& filter : web_options.filters) {
+    options->filters.push_back(blink::mojom::WebBluetoothScanFilter::From<
+                               blink::WebBluetoothScanFilter>(filter));
+  }
+  for (const auto& optional_service : web_options.optionalServices) {
+    options->optional_services.push_back(
+        device::BluetoothUUID(optional_service.utf8()));
+  }
   return options;
 }
 
 // static
-base::Optional<device::BluetoothUUID>
-TypeConverter<base::Optional<device::BluetoothUUID>, blink::WebString>::Convert(
+device::BluetoothUUID
+TypeConverter<device::BluetoothUUID, blink::WebString>::Convert(
     const blink::WebString& web_string) {
-  base::Optional<device::BluetoothUUID> uuid =
-      device::BluetoothUUID(web_string.utf8());
+  device::BluetoothUUID uuid = device::BluetoothUUID(web_string.utf8());
 
-  DCHECK(uuid->IsValid());
+  DCHECK(uuid.IsValid());
 
   return uuid;
 }
 
 }  // namespace mojo