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

Side by Side Diff: chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc

Issue 655783002: Attempt to fix flaky ChromeOS test timeouts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 "base/basictypes.h" 5 #include "base/basictypes.h"
6 #include "base/compiler_specific.h" 6 #include "base/compiler_specific.h"
7 #include "base/prefs/pref_service.h" 7 #include "base/prefs/pref_service.h"
8 #include "base/strings/stringprintf.h" 8 #include "base/strings/stringprintf.h"
9 #include "chrome/browser/chromeos/login/login_manager_test.h" 9 #include "chrome/browser/chromeos/login/login_manager_test.h"
10 #include "chrome/browser/chromeos/login/startup_utils.h" 10 #include "chrome/browser/chromeos/login/startup_utils.h"
(...skipping 280 matching lines...) Expand 10 before | Expand all | Expand 10 after
291 RegisterUser(kTestOwner); 291 RegisterUser(kTestOwner);
292 RegisterUser(kTestNonOwner); 292 RegisterUser(kTestNonOwner);
293 StartupUtils::MarkOobeCompleted(); 293 StartupUtils::MarkOobeCompleted();
294 } 294 }
295 295
296 // Tests the shared setting indicator for the primary user's auto-lock setting 296 // Tests the shared setting indicator for the primary user's auto-lock setting
297 // when the secondary user has enabled or disabled their preference. 297 // when the secondary user has enabled or disabled their preference.
298 // (The checkbox is unset if the current user's preference is false, but if any 298 // (The checkbox is unset if the current user's preference is false, but if any
299 // other signed-in user has enabled this preference, the shared setting 299 // other signed-in user has enabled this preference, the shared setting
300 // indicator explains this.) 300 // indicator explains this.)
301 // Disabled due to flaky timeouts. https://crbug.com/423053 301 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, ScreenLockPreferencePrimary) {
302 IN_PROC_BROWSER_TEST_F(SharedOptionsTest,
303 DISABLED_ScreenLockPreferencePrimary) {
304 LoginUser(kTestOwner); 302 LoginUser(kTestOwner);
305 UserAddingScreen::Get()->Start(); 303 UserAddingScreen::Get()->Start();
306 content::RunAllPendingInMessageLoop(); 304 content::RunAllPendingInMessageLoop();
307 AddUser(kTestNonOwner); 305 AddUser(kTestNonOwner);
308 306
309 user_manager::UserManager* manager = user_manager::UserManager::Get(); 307 user_manager::UserManager* manager = user_manager::UserManager::Get();
310 const user_manager::User* user1 = manager->FindUser(kTestOwner); 308 const user_manager::User* user1 = manager->FindUser(kTestOwner);
311 const user_manager::User* user2 = manager->FindUser(kTestNonOwner); 309 const user_manager::User* user2 = manager->FindUser(kTestNonOwner);
312 310
313 PrefService* prefs1 = 311 PrefService* prefs1 =
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
426 content::WaitForLoadStop(contents); 424 content::WaitForLoadStop(contents);
427 { 425 {
428 SCOPED_TRACE("Screen lock false for both users"); 426 SCOPED_TRACE("Screen lock false for both users");
429 expected_value = false; 427 expected_value = false;
430 CheckBooleanPreference(contents, prefs::kEnableAutoScreenLock, disabled, 428 CheckBooleanPreference(contents, prefs::kEnableAutoScreenLock, disabled,
431 empty_controlled, expected_value); 429 empty_controlled, expected_value);
432 } 430 }
433 } 431 }
434 432
435 } // namespace chromeos 433 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/preferences_browsertest.cc ('k') | chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698