Fixed a bug in the WaitUntilNavigationCompletes automation command.

chrome/browser/automation/testing_automation_provider.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/automation/testing_automation_provider.h"
 
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 6070 matching lines @@
   Browser* browser;
   WebContents* web_contents;
   if (!GetBrowserAndTabFromJSONArgs(args, &browser, &web_contents, &error)) {
     AutomationJSONReply(this, reply_message).SendError(error);
     return;
   }
   NavigationNotificationObserver* observer =
       new NavigationNotificationObserver(&web_contents->GetController(), this,
                                          reply_message, 1, true, true);
   if (!web_contents->IsLoading()) {
-    AutomationJSONReply(this, reply_message).SendSuccess(NULL);
-    delete observer;
+    observer->ConditionMet(AUTOMATION_MSG_NAVIGATION_SUCCESS);
     return;
   }
 }
 
 void TestingAutomationProvider::ExecuteJavascriptJSON(
     DictionaryValue* args,
     IPC::Message* reply_message) {
   if (SendErrorIfModalDialogActive(this, reply_message))
     return;
 
@@ skipping 565 matching lines @@
 }
 
 void TestingAutomationProvider::EnsureTabSelected(Browser* browser,
                                                   WebContents* tab) {
   if (browser->GetActiveWebContents() != tab ||
       browser != BrowserList::GetLastActive()) {
     browser->ActivateTabAt(browser->GetIndexOfController(&tab->GetController()),
         true /* user_gesture */);
   }
 }