Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(61)

Unified Diff: chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc

Issue 180163009: chrome.bluetooth API improvements. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix ChromeOS Full build. Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc
diff --git a/chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc b/chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc
index 6c6c23c8d1ae666d50d30ac1dec5dcc04be74a37..2732bea178317a295117c887ab9379d6bfeef54c 100644
--- a/chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc
+++ b/chrome/browser/extensions/api/bluetooth/bluetooth_extension_function.cc
@@ -7,9 +7,12 @@
#include "base/memory/ref_counted.h"
#include "chrome/browser/extensions/api/bluetooth/bluetooth_api.h"
#include "chrome/browser/extensions/api/bluetooth/bluetooth_event_router.h"
+#include "content/public/browser/browser_thread.h"
#include "device/bluetooth/bluetooth_adapter.h"
#include "device/bluetooth/bluetooth_adapter_factory.h"
+using content::BrowserThread;
+
namespace {
const char kPlatformNotSupported[] =
@@ -17,22 +20,24 @@ const char kPlatformNotSupported[] =
extensions::BluetoothEventRouter* GetEventRouter(
content::BrowserContext* context) {
- return extensions::BluetoothAPI::Get(context)->bluetooth_event_router();
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ return extensions::BluetoothAPI::Get(context)->event_router();
}
bool IsBluetoothSupported(content::BrowserContext* context) {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
return GetEventRouter(context)->IsBluetoothSupported();
}
void GetAdapter(const device::BluetoothAdapterFactory::AdapterCallback callback,
content::BrowserContext* context) {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
GetEventRouter(context)->GetAdapter(callback);
}
} // namespace
namespace extensions {
-
namespace api {
BluetoothExtensionFunction::BluetoothExtensionFunction() {
@@ -42,6 +47,8 @@ BluetoothExtensionFunction::~BluetoothExtensionFunction() {
}
bool BluetoothExtensionFunction::RunImpl() {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
if (!IsBluetoothSupported(browser_context())) {
SetError(kPlatformNotSupported);
return false;
@@ -54,9 +61,9 @@ bool BluetoothExtensionFunction::RunImpl() {
void BluetoothExtensionFunction::RunOnAdapterReady(
scoped_refptr<device::BluetoothAdapter> adapter) {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
DoWork(adapter);
}
} // namespace api
-
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698