bluetooth: Avoid including non-blink mojo bindings in blink code

third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.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 "modules/bluetooth/BluetoothRemoteGATTCharacteristic.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMDataView.h"
@@ skipping 104 matching lines @@
         if (!m_resolver->getExecutionContext() || m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
             return;
 
         DOMDataView* domDataView = ConvertWebVectorToDataView(value);
         if (m_webCharacteristic)
             m_webCharacteristic->setValue(domDataView);
 
         m_resolver->resolve(domDataView);
     }
 
-    void onError(const WebBluetoothError& e) override
+    void onError(int32_t error) override

[Jeffrey Yasskin, 2016/07/13 20:34:58]

Please comment which enum this expects to receive.

[ortuno, 2016/07/13 22:47:29]

Done.

     {
         if (!m_resolver->getExecutionContext() || m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
             return;
-        m_resolver->reject(BluetoothError::take(m_resolver, e));
+        m_resolver->reject(BluetoothError::take(m_resolver, error));
     }
 
 private:
     WeakPersistent<BluetoothRemoteGATTCharacteristic> m_webCharacteristic;
     Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 ScriptPromise BluetoothRemoteGATTCharacteristic::readValue(ScriptState* scriptState)
 {
     WebBluetooth* webbluetooth = BluetoothSupplement::fromScriptState(scriptState);
@@ skipping 13 matching lines @@
     {
         if (!m_resolver->getExecutionContext() || m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
             return;
 
         if (m_webCharacteristic) {
             m_webCharacteristic->setValue(ConvertWebVectorToDataView(value));
         }
         m_resolver->resolve();
     }
 
-    void onError(const WebBluetoothError& e) override
+    void onError(int32_t error) override
     {
         if (!m_resolver->getExecutionContext() || m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
             return;
-        m_resolver->reject(BluetoothError::take(m_resolver, e));
+        m_resolver->reject(BluetoothError::take(m_resolver, error));
     }
 
 private:
     WeakPersistent<BluetoothRemoteGATTCharacteristic> m_webCharacteristic;
     Persistent<ScriptPromiseResolver> m_resolver;
 };
 
 ScriptPromise BluetoothRemoteGATTCharacteristic::writeValue(ScriptState* scriptState, const DOMArrayPiece& value)
 {
     WebBluetooth* webbluetooth = BluetoothSupplement::fromScriptState(scriptState);
@@ skipping 46 matching lines @@
 DEFINE_TRACE(BluetoothRemoteGATTCharacteristic)
 {
     visitor->trace(m_service);
     visitor->trace(m_properties);
     visitor->trace(m_value);
     EventTargetWithInlineData::trace(visitor);
     ActiveDOMObject::trace(visitor);
 }
 
 } // namespace blink