bluetooth: Blink-side implementation of getCharacteristic.

Source/modules/bluetooth/BluetoothGATTCharacteristic.cpp

+// Copyright 2015 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 "config.h"
+#include "modules/bluetooth/BluetoothGATTCharacteristic.h"
+
+#include "bindings/core/v8/CallbackPromiseAdapter.h"
+#include "bindings/core/v8/ScriptPromise.h"
+#include "bindings/core/v8/ScriptPromiseResolver.h"
+#include "core/dom/DOMException.h"
+#include "core/dom/ExceptionCode.h"
+#include "modules/bluetooth/BluetoothError.h"
+#include "public/platform/Platform.h"
+#include "public/platform/modules/bluetooth/WebBluetooth.h"
+#include "wtf/OwnPtr.h"
+
+namespace blink {
+
+BluetoothGATTCharacteristic::BluetoothGATTCharacteristic(const WebBluetoothGATTCharacteristic& webCharacteristic)
+    : m_webCharacteristic(webCharacteristic)
+{
+}
+
+BluetoothGATTCharacteristic* BluetoothGATTCharacteristic::take(ScriptPromiseResolver*, WebBluetoothGATTCharacteristic* webCharacteristicRawPointer)
+{
+    if (!webCharacteristicRawPointer) {
+        return nullptr;
+    }
+    OwnPtr<WebBluetoothGATTCharacteristic> webCharacteristic = adoptPtr(webCharacteristicRawPointer);
+    return new BluetoothGATTCharacteristic(*webCharacteristic);

[Jeffrey Yasskin, 2015/06/03 00:24:04]

This is a little weird (was probably weird in your other patch too, but I didn't
look as closely). Why is it right to copy the WebBluetoothGATTCharacteristic
into the BluetoothGATTCharacteristic instead of having
BluetoothGATTCharacteristic store an OwnPtr<WebBluetoothGATTCharacteristic> and
just take ownership of the already-allocated one?

[ortuno, 2015/06/03 21:57:04]

Changed to PassOwnPtr. OwnPtr alone won't compile.

+}
+
+void BluetoothGATTCharacteristic::dispose(WebBluetoothGATTCharacteristic* webCharacteristic)
+{
+    delete webCharacteristic;
+}
+
+} // namespace blink