bluetooth: Enable Web Bluetooth in Origin Trials

third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp

Index: third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp
diff --git a/third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp b/third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp
index 55c97f33251159e4c3212e0430aab1f55b48dd37..00ace3c6760423696c40eab14ab16eb7d07945e4 100644
--- a/third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp
+++ b/third_party/WebKit/Source/modules/bluetooth/Bluetooth.cpp
@@ -130,6 +130,18 @@ static void convertRequestDeviceOptions(const RequestDeviceOptions& options, Web
 // https://webbluetoothchrome.github.io/web-bluetooth/#dom-bluetooth-requestdevice
 ScriptPromise Bluetooth::requestDevice(ScriptState* scriptState, const RequestDeviceOptions& options, ExceptionState& exceptionState)
 {
+
+    // By adding the "OriginTrialEnabled" extended binding, we enable the
+    // requestDevice function on all platforms for websites that contain an
+    // origin trial token. Since we only support Chrome OS, Android and macOS

[iclelland, 2016/06/24 18:36:32]

nit: MacOS

[ortuno, 2016/06/24 19:16:33]

I think it's macOS, like in iOS O_O.
https://9to5mac.files.wordpress.com/2016/06/wwdc-2016-watchos-tvos-macos-ios....

[iclelland, 2016/06/24 19:46:35]

Maybe --or Mac OS X, or MacOSX, or just 'OS X' ;)
(Even more authoritatively,
https://web.archive.org/web/20160624143934/https://developer.apple.com/macos/ )

I went for consistency in the codebase -- 702 results at
https://cs.chromium.org/search/?q=MacOS+case:yes&sq=package:chromium&type=cs
versus 3 for
https://cs.chromium.org/search/?q=macOS+case:yes&sq=package:chromium&type=cs

But it's a nit, so I'm really not going to rescind an l g t m over it :)

[ortuno, 2016/06/27 20:22:42]

Ah I guess we are not using the new name yet. I'll change to match the rest of
the codebase.

+    // for this experiment we reject any promises from other platforms unless
+    // they have the enable-web-bluetooth flag on.
+#if !OS(CHROMEOS) && !OS(ANDROID) && !OS(MACOSX)
+    if (!RuntimeEnabledFeatures::webBluetoothEnabled()) {
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError, "Web Bluetooth is not enabled on this platform. To find out how to enable it and the current implementation status visit https://goo.gl/HKa2If"));
+    }
+#endif
+
     // 1. If the incumbent settings object is not a secure context, reject promise with a SecurityError and abort these steps.
     String errorMessage;
     if (!scriptState->getExecutionContext()->isSecureContext(errorMessage)) {