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

Unified Diff: chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc

Issue 2166503003: Fix bluetooth adapter access in SystemTrayDelegateChromeOS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Do not check on InputMethodManager::Get. Created 4 years, 5 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/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc
diff --git a/chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc b/chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc
index f57079ab841fdccbaecdd792569b1f723ad6fad0..935c863da67ff15ea71077c21eb46fe73d3a5dc7 100644
--- a/chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc
+++ b/chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc
@@ -125,6 +125,14 @@ IN_PROC_BROWSER_TEST_F(SystemTrayDelegateChromeOSTest,
EXPECT_EQ(base::k24HourClock, GetHourType());
}
+// Test that checking bluetooth status early on doesn't cause a crash.
+IN_PROC_BROWSER_TEST_F(SystemTrayDelegateChromeOSTest, BluetoothStatus) {
James Cook 2016/07/20 15:23:14 drive-by question: Any particular reason you added
msw 2016/07/20 16:43:19 Woops, I meant to delete this; thanks for catching
+ SystemTrayDelegateChromeOS delegate;
+ EXPECT_FALSE(delegate.GetBluetoothAvailable());
+ EXPECT_FALSE(delegate.GetBluetoothEnabled());
+ EXPECT_FALSE(delegate.IsBluetoothDiscovering());
+}
+
// Makes sure that no notifications are shown when rotating the
// display on display settings URLs.
IN_PROC_BROWSER_TEST_F(DisplayNotificationsTest,

Powered by Google App Engine
This is Rietveld 408576698