Expose SetDiscoveryFilter from BluetoothEventRouter

extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc

Index: extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc
diff --git a/extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc b/extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc
index 7f5a41283cdfb38d79fe2163020f092cf6dbaf0c..8ca5d388ad93421db5093a5359bb0098d2de6990 100644
--- a/extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc
+++ b/extensions/browser/api/bluetooth/bluetooth_event_router_unittest.cc
@@ -27,6 +27,9 @@ const char kTestExtensionId[] = "test extension id";
 const device::BluetoothUUID kAudioProfileUuid("1234");
 const device::BluetoothUUID kHealthProfileUuid("4321");
 
+MATCHER_P(IsFilterEqual, a, "") {
+  return arg.Equals(*a);
+}
 }  // namespace
 
 namespace extensions {
@@ -91,4 +94,33 @@ TEST_F(BluetoothEventRouterTest, UnloadExtension) {
   EXPECT_CALL(*mock_adapter_, RemoveObserver(testing::_)).Times(1);
 }
 
+// This test check that calling SetDiscoveryFitler before StartDiscoverySession

[armansito, 2015/04/16 19:13:55]

nit: s/SetDiscoveryFitler/SetDiscoveryFilter/

Note: Fitler sounds like Hitler.

[jpawlowski1, 2015/04/16 19:36:28]

Done.

+// for given extension will start session with proper filter.
+TEST_F(BluetoothEventRouterTest, SetDiscoveryFilter) {
+  scoped_ptr<device::BluetoothDiscoveryFilter> discovery_filter(
+      new device::BluetoothDiscoveryFilter(
+          device::BluetoothDiscoveryFilter::Transport::TRANSPORT_LE));
+
+  discovery_filter->SetRSSI(-80);
+  discovery_filter->AddUUID(device::BluetoothUUID("1000"));
+
+  device::BluetoothDiscoveryFilter df(
+      device::BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+  df.CopyFrom(*discovery_filter);
+
+  router_->SetDiscoveryFilter(discovery_filter.Pass(), mock_adapter_,
+                              kTestExtensionId, base::Bind(&base::DoNothing),
+                              base::Bind(&base::DoNothing));
+
+  EXPECT_CALL(*mock_adapter_, StartDiscoverySessionWithFilterRaw(
+                                  testing::Pointee(IsFilterEqual(&df)),
+                                  testing::_, testing::_)).Times(1);
+
+  router_->StartDiscoverySession(mock_adapter_, kTestExtensionId,
+                                 base::Bind(&base::DoNothing),
+                                 base::Bind(&base::DoNothing));
+
+  EXPECT_CALL(*mock_adapter_, RemoveObserver(testing::_)).Times(1);

[armansito, 2015/04/16 19:13:55]

You'll also want test cases to bluetooth_private_apitest.cc that exercise the
actual JS code using your mocks.

[jpawlowski1, 2015/04/16 19:36:27]

this code is not yet connected to bluetooth private api. Next CL will calling
EventRouter from bluetooth_private_api together with tests.

+}
+
 }  // namespace extensions