[Async][WIP] Call FrameLoader::checkCompleted() asynchronously to avoid sync body.onload()

chrome/browser/extensions/extension_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 "chrome/browser/extensions/extension_browsertest.h"
 
 #include <stddef.h>
 #include <utility>
 #include <vector>
 
@@ skipping 569 matching lines @@
       profile())->extension_service();
   service->EnableExtension(extension_id);
 }
 
 void ExtensionBrowserTest::OpenWindow(content::WebContents* contents,
                                       const GURL& url,
                                       bool newtab_process_should_equal_opener,
                                       content::WebContents** newtab_result) {
   content::WindowedNotificationObserver windowed_observer(
       content::NOTIFICATION_LOAD_STOP,
-      content::NotificationService::AllSources());
+      content::Source<content::NavigationController>(&contents->GetController()));
   ASSERT_TRUE(content::ExecuteScript(contents,
                                      "window.open('" + url.spec() + "');"));
 
   // The above window.open call is not user-initiated, so it will create
   // a popup window instead of a new tab in current window.
   // The stop notification will come from the new tab.
   windowed_observer.Wait();
   content::NavigationController* controller =
       content::Source<content::NavigationController>(
           windowed_observer.source()).ptr();
@@ skipping 10 matching lines @@
 }
 
 void ExtensionBrowserTest::NavigateInRenderer(content::WebContents* contents,
                                               const GURL& url) {
   // Ensure any existing navigations complete before trying to navigate anew, to
   // avoid triggering of the unload event for the wrong navigation.
   content::WaitForLoadStop(contents);
   bool result = false;
   content::WindowedNotificationObserver windowed_observer(
       content::NOTIFICATION_LOAD_STOP,
-      content::NotificationService::AllSources());
+      content::Source<content::NavigationController>(&contents->GetController()));
+      //content::NotificationService::AllSources());
   ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
       contents,
       "window.addEventListener('unload', function() {"
       "    window.domAutomationController.send(true);"
       "}, false);"
       "window.location = '" + url.spec() + "';",
       &result));
   ASSERT_TRUE(result);
   windowed_observer.Wait();
   EXPECT_EQ(url, contents->GetController().GetLastCommittedEntry()->GetURL());
@@ skipping 22 matching lines @@
   return extensions::browsertest_util::ExecuteScriptInBackgroundPage(
       profile(), extension_id, script);
 }
 
 bool ExtensionBrowserTest::ExecuteScriptInBackgroundPageNoWait(
     const std::string& extension_id,
     const std::string& script) {
   return extensions::browsertest_util::ExecuteScriptInBackgroundPageNoWait(
       profile(), extension_id, script);
 }