Add PyAuto test for DeviceGuestModeEnabled policy

chrome/browser/ui/webui/chromeos/login/signin_screen_handler.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 "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
@@ skipping 898 matching lines @@
     const base::ListValue* args) {
   is_account_picker_showing_first_time_ = true;
   MaybePreloadAuthExtension();
 
   if (ScreenLocker::default_screen_locker()) {
     content::NotificationService::current()->Notify(
         chrome::NOTIFICATION_LOCK_WEBUI_READY,
         content::NotificationService::AllSources(),
         content::NotificationService::NoDetails());
   }
+
+  content::NotificationService::current()->Notify(

[Nikita (slow), 2012/05/16 15:56:13]

I don't think that you need that new notification, see another comment in this
file.

[bartfab (slow), 2012/05/16 18:07:10]

Done.

+      chrome::NOTIFICATION_ACCOUNT_PICKER_WEBUI_READY,
+      content::NotificationService::AllSources(),
+      content::NotificationService::NoDetails());
 }
 
 void SigninScreenHandler::HandleLoginWebuiReady(const base::ListValue* args) {
   if (focus_stolen_) {
     // Set focus to the Gaia page.
     // TODO(altimofeev): temporary solution, until focus parameters are
     // implemented on the Gaia side.
     // Do this only once. Any subsequent call would relod GAIA frame.
     focus_stolen_ = false;
     const char code[] = "gWindowOnLoad();";
@@ skipping 53 matching lines @@
 }
 
 void SigninScreenHandler::HandleSignOutUser(const base::ListValue* args) {
   if (!delegate_)
     return;
   delegate_->Signout();
 }
 
 void SigninScreenHandler::HandleUserImagesLoaded(const base::ListValue* args) {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_LOGIN_USER_IMAGES_LOADED,

[Nikita (slow), 2012/05/16 15:56:13]

This one is almost the same notification. It is called a bit later, when user
images are loaded.

[bartfab (slow), 2012/05/16 18:07:10]

Done.

       content::NotificationService::AllSources(),
       content::NotificationService::NoDetails());
 }
 
 void SigninScreenHandler::HandleNetworkErrorShown(const base::ListValue* args) {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_LOGIN_NETWORK_ERROR_SHOWN,
       content::NotificationService::AllSources(),
       content::NotificationService::NoDetails());
 }
@@ skipping 47 matching lines @@
       !dns_clear_task_running_ &&
       network_state_informer_->is_online()) {
     gaia_silent_load_ = true;
     gaia_silent_load_network_ =
         network_state_informer_->active_network_service_path();
     LoadAuthExtension(true, true, false);
   }
 }
 
 }  // namespace chromeos