Artifically delay the load finished signal until after dom content loaded

chrome/browser/extensions/api/web_navigation/web_navigation_api.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.
 
 // Implements the Chrome Extensions WebNavigation API.
 
 #include "chrome/browser/extensions/api/web_navigation/web_navigation_api.h"
 
 #include "base/lazy_instance.h"
 #include "chrome/browser/extensions/api/web_navigation/web_navigation_api_constants.h"
@@ skipping 510 matching lines @@
     int64 frame_num,
     content::RenderViewHost* render_view_host) {
   DVLOG(2) << "DocumentLoadedInFrame("
            << "render_view_host=" << render_view_host
            << ", frame_num=" << frame_num << ")";
   if (render_view_host != render_view_host_)
     return;
   FrameNavigationState::FrameID frame_id(frame_num, render_view_host);
   if (!navigation_state_.CanSendEvents(frame_id))
     return;
+  navigation_state_.SetParsingFinished(frame_id);
   helpers::DispatchOnDOMContentLoaded(web_contents(),
                                       navigation_state_.GetUrl(frame_id),
                                       navigation_state_.IsMainFrame(frame_id),
                                       frame_num);
+
+  if (!navigation_state_.GetNavigationCompleted(frame_id))
+    return;
+
+  // The load might already have finished by the time we finished parsing. For
+  // compatibility reasons, we artifically delay the load completed signal until
+  // after parsing was completed.
+  helpers::DispatchOnCompleted(web_contents(),
+                               navigation_state_.GetUrl(frame_id),
+                               navigation_state_.IsMainFrame(frame_id),
+                               frame_num);
 }
 
 void WebNavigationTabObserver::DidFinishLoad(
     int64 frame_num,
     const GURL& validated_url,
     bool is_main_frame,
     content::RenderViewHost* render_view_host) {
   DVLOG(2) << "DidFinishLoad("
            << "render_view_host=" << render_view_host
            << ", frame_num=" << frame_num
            << ", url=" << validated_url << ")";
   if (render_view_host != render_view_host_)
     return;
   FrameNavigationState::FrameID frame_id(frame_num, render_view_host);
   // When showing replacement content, we might get load signals for frames
   // that weren't reguarly loaded.
   if (!navigation_state_.IsValidFrame(frame_id))
     return;
   navigation_state_.SetNavigationCompleted(frame_id);
   if (!navigation_state_.CanSendEvents(frame_id))
     return;
   DCHECK(navigation_state_.GetUrl(frame_id) == validated_url ||
          (navigation_state_.GetUrl(frame_id) == GURL(chrome::kAboutSrcDocURL) &&
           validated_url == GURL(chrome::kAboutBlankURL)))
       << "validated URL is " << validated_url << " but we expected "
       << navigation_state_.GetUrl(frame_id);
   DCHECK_EQ(navigation_state_.IsMainFrame(frame_id), is_main_frame);
+
+  // The load might already have finished by the time we finished parsing. For
+  // compatibility reasons, we artifically delay the load completed signal until
+  // after parsing was completed.
+  if (!navigation_state_.GetParsingFinished(frame_id))
+    return;
   helpers::DispatchOnCompleted(web_contents(),
                                navigation_state_.GetUrl(frame_id),
                                is_main_frame,
                                frame_num);
 }
 
 void WebNavigationTabObserver::DidFailLoad(
     int64 frame_num,
     const GURL& validated_url,
     bool is_main_frame,
@@ skipping 261 matching lines @@
 WebNavigationAPI::GetFactoryInstance() {
   return &g_factory.Get();
 }
 
 void WebNavigationAPI::OnListenerAdded(const EventListenerInfo& details) {
   web_navigation_event_router_.reset(new WebNavigationEventRouter(profile_));
   ExtensionSystem::Get(profile_)->event_router()->UnregisterObserver(this);
 }
 
 }  // namespace extensions