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

Side by Side Diff: device/bluetooth/bluetooth_adapter_unittest.cc

Issue 2032273002: If location services are turned off, have the BT chooser prompt the user to turn them on. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: Sync Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "device/bluetooth/bluetooth_adapter.h" 5 #include "device/bluetooth/bluetooth_adapter.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 540 matching lines...) Expand 10 before | Expand all | Expand 10 after
551 return; 551 return;
552 } 552 }
553 553
554 StartLowEnergyDiscoverySessionExpectedToFail(); 554 StartLowEnergyDiscoverySessionExpectedToFail();
555 555
556 EXPECT_EQ(0, callback_count_); 556 EXPECT_EQ(0, callback_count_);
557 EXPECT_EQ(1, error_callback_count_); 557 EXPECT_EQ(1, error_callback_count_);
558 } 558 }
559 #endif // defined(OS_ANDROID) || defined(OS_MACOSX) 559 #endif // defined(OS_ANDROID) || defined(OS_MACOSX)
560 560
561 // Android-only: Only Android requires location services to be turned on to scan
562 // for Bluetooth devices.
563 #if defined(OS_ANDROID)
564 // Checks that discovery fails (instead of hanging) when location services are
565 // turned off.
566 TEST_F(BluetoothTest, NoLocationServices) {
567 if (!PlatformSupportsLowEnergy()) {
568 LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test.";
569 return;
570 }
571 InitWithFakeAdapter();
572 TestBluetoothAdapterObserver observer(adapter_);
573
574 SimulateLocationServicesOff();
575
576 StartLowEnergyDiscoverySessionExpectedToFail();
577
578 EXPECT_EQ(0, callback_count_);
579 EXPECT_EQ(1, error_callback_count_);
580 }
581 #endif // defined(OS_ANDROID)
582
561 #if defined(OS_ANDROID) || defined(OS_MACOSX) || defined(OS_WIN) 583 #if defined(OS_ANDROID) || defined(OS_MACOSX) || defined(OS_WIN)
562 // Discovers a device. 584 // Discovers a device.
563 TEST_F(BluetoothTest, DiscoverLowEnergyDevice) { 585 TEST_F(BluetoothTest, DiscoverLowEnergyDevice) {
564 if (!PlatformSupportsLowEnergy()) { 586 if (!PlatformSupportsLowEnergy()) {
565 LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test."; 587 LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test.";
566 return; 588 return;
567 } 589 }
568 InitWithFakeAdapter(); 590 InitWithFakeAdapter();
569 TestBluetoothAdapterObserver observer(adapter_); 591 TestBluetoothAdapterObserver observer(adapter_);
570 592
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
763 service->Register(GetCallback(Call::NOT_EXPECTED), 785 service->Register(GetCallback(Call::NOT_EXPECTED),
764 GetGattErrorCallback(Call::EXPECTED)); 786 GetGattErrorCallback(Call::EXPECTED));
765 service->Unregister(GetCallback(Call::EXPECTED), 787 service->Unregister(GetCallback(Call::EXPECTED),
766 GetGattErrorCallback(Call::NOT_EXPECTED)); 788 GetGattErrorCallback(Call::NOT_EXPECTED));
767 service->Unregister(GetCallback(Call::NOT_EXPECTED), 789 service->Unregister(GetCallback(Call::NOT_EXPECTED),
768 GetGattErrorCallback(Call::EXPECTED)); 790 GetGattErrorCallback(Call::EXPECTED));
769 } 791 }
770 #endif // defined(OS_CHROMEOS) || defined(OS_LINUX) 792 #endif // defined(OS_CHROMEOS) || defined(OS_LINUX)
771 793
772 } // namespace device 794 } // namespace device
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698