Bluetooth: remove Profile API

chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc

 // Copyright (c) 2012 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 <string.h>
 
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/api/bluetooth/bluetooth_api.h"
 #include "chrome/browser/extensions/api/bluetooth/bluetooth_event_router.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_function_test_utils.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 #include "device/bluetooth/bluetooth_uuid.h"
 #include "device/bluetooth/test/mock_bluetooth_adapter.h"
 #include "device/bluetooth/test/mock_bluetooth_device.h"
 #include "device/bluetooth/test/mock_bluetooth_discovery_session.h"
-#include "device/bluetooth/test/mock_bluetooth_profile.h"
-#include "device/bluetooth/test/mock_bluetooth_socket.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using device::BluetoothAdapter;
 using device::BluetoothDevice;
 using device::BluetoothDiscoverySession;
-using device::BluetoothProfile;
 using device::BluetoothUUID;
 using device::MockBluetoothAdapter;
 using device::MockBluetoothDevice;
 using device::MockBluetoothDiscoverySession;
-using device::MockBluetoothProfile;
 using extensions::Extension;
 
 namespace utils = extension_function_test_utils;
 namespace api = extensions::api;
 
 namespace {
 
 static const char* kAdapterAddress = "A1:A2:A3:A4:A5:A6";
 static const char* kName = "whatsinaname";
 
 class BluetoothApiTest : public ExtensionApiTest {
  public:
   BluetoothApiTest() {}
 
   virtual void SetUpOnMainThread() OVERRIDE {
     ExtensionApiTest::SetUpOnMainThread();
     empty_extension_ = utils::CreateEmptyExtension();
     SetUpMockAdapter();
-    profile1_.reset(new testing::NiceMock<MockBluetoothProfile>());
-    profile2_.reset(new testing::NiceMock<MockBluetoothProfile>());
   }
 
   virtual void CleanUpOnMainThread() OVERRIDE {
     EXPECT_CALL(*mock_adapter_, RemoveObserver(testing::_));
   }
 
   void SetUpMockAdapter() {
     // The browser will clean this up when it is torn down
     mock_adapter_ = new testing::StrictMock<MockBluetoothAdapter>();
     event_router()->SetAdapterForTest(mock_adapter_);
@@ skipping 26 matching lines @@
     function->set_has_callback(true);
     return function;
   }
 
  protected:
   testing::StrictMock<MockBluetoothAdapter>* mock_adapter_;
   scoped_ptr<testing::NiceMock<MockBluetoothDiscoverySession> > mock_session_;
   scoped_ptr<testing::NiceMock<MockBluetoothDevice> > device1_;
   scoped_ptr<testing::NiceMock<MockBluetoothDevice> > device2_;
   scoped_ptr<testing::NiceMock<MockBluetoothDevice> > device3_;
-  scoped_ptr<testing::NiceMock<MockBluetoothProfile> > profile1_;
-  scoped_ptr<testing::NiceMock<MockBluetoothProfile> > profile2_;
 
   extensions::BluetoothEventRouter* event_router() {
     return bluetooth_api()->event_router();
   }
 
   extensions::BluetoothAPI* bluetooth_api() {
     return extensions::BluetoothAPI::Get(browser()->profile());
   }
 
  private:
   scoped_refptr<Extension> empty_extension_;
 };
 
-class TestBluetoothAddProfileFunction
-    : public api::BluetoothAddProfileFunction {
- public:
-  explicit TestBluetoothAddProfileFunction(BluetoothProfile* profile)
-      : BluetoothAddProfileFunction(), profile_(profile) {
-  }
-
- protected:
-  virtual ~TestBluetoothAddProfileFunction() {
-  }
-
-  // BluetoothAddProfileFunction override.
-  virtual void RegisterProfile(
-      const device::BluetoothProfile::Options& options,
-      const device::BluetoothProfile::ProfileCallback& callback) OVERRIDE {
-    callback.Run(profile_);
-  }
-
- private:
-  // TestBluetoothAddProfileFunction does not own |profile_|.
-  BluetoothProfile* profile_;
-};
-
 static void StopDiscoverySessionCallback(const base::Closure& callback,
                                          const base::Closure& error_callback) {
   callback.Run();
 }
 
 }  // namespace
 
-IN_PROC_BROWSER_TEST_F(BluetoothApiTest, Profiles) {
-  // Run in context of an extension that has permissions for the profiles
-  // we intend to register.
-  scoped_refptr<const Extension> extension(
-      LoadExtension(test_data_dir_.AppendASCII("bluetooth/profiles")));
-  ASSERT_TRUE(extension.get());
-
-  EXPECT_CALL(*profile1_, SetConnectionCallback(testing::_));
-  scoped_refptr<TestBluetoothAddProfileFunction> add_profile_function;
-  add_profile_function = new TestBluetoothAddProfileFunction(profile1_.get());
-  add_profile_function->set_extension(extension.get());
-  add_profile_function->set_has_callback(true);
-  std::string error(utils::RunFunctionAndReturnError(
-      add_profile_function.get(), "[{\"uuid\": \"1234\"}]", browser()));
-  ASSERT_TRUE(error.empty());
-
-  // Registering the profile for the same uuid again will throw an error.
-  add_profile_function = new TestBluetoothAddProfileFunction(profile2_.get());
-  add_profile_function->set_extension(extension.get());
-  add_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      add_profile_function.get(), "[{\"uuid\": \"1234\"}]", browser());
-  ASSERT_FALSE(error.empty());
-
-  add_profile_function = new TestBluetoothAddProfileFunction(profile2_.get());
-  add_profile_function->set_extension(extension.get());
-  add_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      add_profile_function.get(), "[{\"uuid\": \"5678\"}]", browser());
-  ASSERT_TRUE(error.empty());
-
-  scoped_refptr<api::BluetoothRemoveProfileFunction> remove_profile_function;
-  remove_profile_function = new api::BluetoothRemoveProfileFunction();
-  remove_profile_function->set_extension(extension.get());
-  remove_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      remove_profile_function.get(), "[{\"uuid\": \"1234\"}]", browser());
-  ASSERT_TRUE(error.empty());
-
-  remove_profile_function = new api::BluetoothRemoveProfileFunction();
-  remove_profile_function->set_extension(extension.get());
-  remove_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      remove_profile_function.get(), "[{\"uuid\": \"5678\"}]", browser());
-  ASSERT_TRUE(error.empty());
-
-  // Removing the same profile again will throw an error.
-  remove_profile_function = new api::BluetoothRemoveProfileFunction();
-  remove_profile_function->set_extension(extension.get());
-  remove_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      remove_profile_function.get(), "[{\"uuid\": \"5678\"}]", browser());
-  ASSERT_FALSE(error.empty());
-
-  // Registering a profile we don't have permission for will throw an error.
-  add_profile_function = new TestBluetoothAddProfileFunction(profile1_.get());
-  add_profile_function->set_extension(extension.get());
-  add_profile_function->set_has_callback(true);
-  error = utils::RunFunctionAndReturnError(
-      add_profile_function.get(), "[{\"uuid\": \"9999\"}]", browser());
-  ASSERT_FALSE(error.empty());
-}
-
 IN_PROC_BROWSER_TEST_F(BluetoothApiTest, GetAdapterState) {
   EXPECT_CALL(*mock_adapter_, GetAddress())
       .WillOnce(testing::Return(kAdapterAddress));
   EXPECT_CALL(*mock_adapter_, GetName())
       .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(false));
   EXPECT_CALL(*mock_adapter_, IsPowered())
       .WillOnce(testing::Return(true));
   EXPECT_CALL(*mock_adapter_, IsDiscovering())
@@ skipping 223 matching lines @@
       .WillOnce(testing::Return(true));
   EXPECT_CALL(*mock_adapter_, IsDiscovering())
       .WillOnce(testing::Return(true));
   event_router()->AdapterDiscoveringChanged(mock_adapter_, true);
 
   listener.Reply("go");
 
   EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }
 
-IN_PROC_BROWSER_TEST_F(BluetoothApiTest, OnConnection) {
-  ResultCatcher catcher;
-  catcher.RestrictToProfile(browser()->profile());
-
-  // Load and wait for setup
-  ExtensionTestMessageListener listener("ready", true);
-  scoped_refptr<const Extension> extension(
-      LoadExtension(test_data_dir_.AppendASCII("bluetooth/on_connection")));
-  ASSERT_TRUE(extension.get());
-  EXPECT_TRUE(listener.WaitUntilSatisfied());
-
-  scoped_refptr<device::MockBluetoothSocket> socket =
-      new device::MockBluetoothSocket();
-
-  EXPECT_CALL(*mock_adapter_, GetDevice(device1_->GetAddress()))
-      .WillOnce(testing::Return(device1_.get()));
-
-  event_router()->AddProfile(
-      BluetoothUUID("1234"), extension->id(), profile1_.get());
-  bluetooth_api()->DispatchConnectionEvent(
-      extension->id(), BluetoothUUID("1234"), device1_.get(), socket);
-  // Connection events are dispatched using a couple of PostTask to the UI
-  // thread. Waiting until idle ensures the event is dispatched to the
-  // receiver(s).
-  base::RunLoop().RunUntilIdle();
-
-  listener.Reply("go");
-  EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
-  event_router()->RemoveProfile(BluetoothUUID("1234"));
-}
-
 IN_PROC_BROWSER_TEST_F(BluetoothApiTest, GetDevices) {
   ResultCatcher catcher;
   catcher.RestrictToProfile(browser()->profile());
 
   BluetoothAdapter::ConstDeviceList devices;
   devices.push_back(device1_.get());
   devices.push_back(device2_.get());
 
   EXPECT_CALL(*mock_adapter_, GetDevices())
       .Times(1)
@@ skipping 82 matching lines @@
   // Load and wait for setup
   ExtensionTestMessageListener listener("ready", true);
   ASSERT_TRUE(
       LoadExtension(test_data_dir_.AppendASCII("bluetooth/device_info")));
   EXPECT_TRUE(listener.WaitUntilSatisfied());
 
   listener.Reply("go");
 
   EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }