Enable TestLoadingPublicUsersFromLocalState

chrome/browser/chromeos/login/existing_user_controller_browsertest.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>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
@@ skipping 442 matching lines @@
       runner1 = new content::MessageLoopRunner;
       subscription1 = chromeos::CrosSettings::Get()->AddSettingsObserver(
           chromeos::kAccountsPrefDeviceLocalAccountAutoLoginId,
           runner1->QuitClosure());
     }
     scoped_refptr<content::MessageLoopRunner> runner2;
     scoped_ptr<CrosSettings::ObserverSubscription> subscription2;
     if (!proto.has_device_local_accounts() ||
         !proto.device_local_accounts().has_auto_login_delay() ||
         proto.device_local_accounts().auto_login_delay() != delay) {
-      runner1 = new content::MessageLoopRunner;
-      subscription1 = chromeos::CrosSettings::Get()->AddSettingsObserver(
+      runner2 = new content::MessageLoopRunner;

[bartfab (slow), 2015/09/08 14:57:17]

Oops, that was quite a stupid typo. Thanks for fixing the test!

+      subscription2 = chromeos::CrosSettings::Get()->AddSettingsObserver(
           chromeos::kAccountsPrefDeviceLocalAccountAutoLoginDelay,
-          runner1->QuitClosure());
+          runner2->QuitClosure());
     }
 
     // Update the policy.
     proto.mutable_device_local_accounts()->set_auto_login_id(username);
     proto.mutable_device_local_accounts()->set_auto_login_delay(delay);
     RefreshDevicePolicy();
 
     // Wait for ExistingUserController to read the updated settings.
     if (runner1.get())
       runner1->Run();
@@ skipping 254 matching lines @@
   // Check that when the timer fires, auto-login fails with an error.
   ExpectLoginFailure();
   FireAutoLogin();
 }
 
 IN_PROC_BROWSER_TEST_F(ExistingUserControllerPublicSessionTest,
                        PRE_TestLoadingPublicUsersFromLocalState) {
   // First run propagates public accounts and stores them in Local State.
 }
 
-// See http://crbug.com/393704; flaky.
 IN_PROC_BROWSER_TEST_F(ExistingUserControllerPublicSessionTest,
-                       DISABLED_TestLoadingPublicUsersFromLocalState) {
+                       TestLoadingPublicUsersFromLocalState) {
   // Second run loads list of public accounts from Local State.
 }
 
 }  // namespace chromeos