Refactored code to allow associating workers with multiple renderers.

SharedWorkers now gracefully handle http auth requests after their
initial window has closed.

BUG=27660
TEST=WorkerHttpAuth,SharedWorkerHttpAuth uitests

Review URL: http://codereview.chromium.org/509016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36888 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 12 deletions

diff --git a/chrome/browser/automation/automation_provider_observers.cc b/chrome/browser/automation/automation_provider_observers.cc
index 03039b1..dfea17c 100644
--- a/chrome/browser/automation/automation_provider_observers.cc
+++ b/chrome/browser/automation/automation_provider_observers.cc
@@ -174,18 +174,16 @@ void NavigationNotificationObserver::Observe(
     // occur while authentication is ongoing.
     navigation_started_ = true;
   } else if (type == NotificationType::AUTH_NEEDED) {
-    if (navigation_started_) {
-      // Remember the login handler that wants authentication.
-      LoginHandler* handler =
-          Details<LoginNotificationDetails>(details)->handler();
-      automation_->AddLoginHandler(controller_, handler);
-
-      // Respond that authentication is needed.
-      navigation_started_ = false;
-      ConditionMet(AUTOMATION_MSG_NAVIGATION_AUTH_NEEDED);
-    } else {
-      NOTREACHED();
-    }
+    // Remember the login handler that wants authentication.
+    // We do this in all cases (not just when navigation_started_ == true) so
+    // tests can still wait for auth dialogs outside of navigation.
+    LoginHandler* handler =
+        Details<LoginNotificationDetails>(details)->handler();
+    automation_->AddLoginHandler(controller_, handler);
+
+    // Respond that authentication is needed.
+    navigation_started_ = false;
+    ConditionMet(AUTOMATION_MSG_NAVIGATION_AUTH_NEEDED);
   } else {
     NOTREACHED();
   }