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

Unified Diff: chrome/browser/automation/automation_provider_observers_chromeos.cc

Issue 10382197: Add PyAuto test for DeviceGuestModeEnabled policy (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Comments addressed. Created 8 years, 7 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
« no previous file with comments | « chrome/browser/automation/automation_provider_observers.h ('k') | chrome/test/functional/PYAUTO_TESTS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/automation_provider_observers_chromeos.cc
diff --git a/chrome/browser/automation/automation_provider_observers_chromeos.cc b/chrome/browser/automation/automation_provider_observers_chromeos.cc
index ead07cbf90d4fa253ec0a41e64c956a118bbbae0..4167ac0579d4d723d49a42accc24834598bb9e39 100644
--- a/chrome/browser/automation/automation_provider_observers_chromeos.cc
+++ b/chrome/browser/automation/automation_provider_observers_chromeos.cc
@@ -48,21 +48,26 @@ void NetworkManagerInitObserver::OnNetworkManagerChanged(NetworkLibrary* obj) {
}
}
-LoginWebuiReadyObserver::LoginWebuiReadyObserver(
- AutomationProvider* automation)
+LoginWebuiReadyObserver::LoginWebuiReadyObserver(AutomationProvider* automation)
: automation_(automation->AsWeakPtr()) {
registrar_.Add(this, chrome::NOTIFICATION_LOGIN_WEBUI_READY,
content::NotificationService::AllSources());
-}
-
-LoginWebuiReadyObserver::~LoginWebuiReadyObserver() {
+ registrar_.Add(this, chrome::NOTIFICATION_LOGIN_USER_IMAGES_LOADED,
+ content::NotificationService::AllSources());
+ registrar_.Add(this, chrome::NOTIFICATION_LOGIN_NETWORK_ERROR_SHOWN,
Nirnimesh 2012/05/16 19:48:26 When is this raised? It takes a while for the logi
Nikita (slow) 2012/05/22 11:09:16 Yes, it is that one. So in that case you'll see NO
+ content::NotificationService::AllSources());
+ registrar_.Add(this, chrome::NOTIFICATION_WIZARD_FIRST_SCREEN_SHOWN,
+ content::NotificationService::AllSources());
}
void LoginWebuiReadyObserver::Observe(
int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {
- DCHECK(type == chrome::NOTIFICATION_LOGIN_WEBUI_READY);
+ DCHECK(type == chrome::NOTIFICATION_LOGIN_WEBUI_READY ||
+ type == chrome::NOTIFICATION_LOGIN_USER_IMAGES_LOADED ||
+ type == chrome::NOTIFICATION_LOGIN_NETWORK_ERROR_SHOWN ||
+ type == chrome::NOTIFICATION_WIZARD_FIRST_SCREEN_SHOWN);
if (automation_)
automation_->OnLoginWebuiReady();
delete this;
« no previous file with comments | « chrome/browser/automation/automation_provider_observers.h ('k') | chrome/test/functional/PYAUTO_TESTS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698