Make LoadMonitoringExtensionHostQueue remove itself as an ExtensionHost observer at the correct tim…

chrome/browser/background/background_contents.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/background/background_contents.h"
 
 #include "base/profiler/scoped_tracker.h"
 #include "chrome/browser/background/background_contents_service.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/chrome_extension_web_contents_observer.h"
@@ skipping 146 matching lines @@
 
   // Our RenderView went away, so we should go away also, so killing the process
   // via the TaskManager doesn't permanently leave a BackgroundContents hanging
   // around the system, blocking future instances from being created
   // <http://crbug.com/65189>.
   delete this;
 }
 
 void BackgroundContents::DidStartLoading(
     content::RenderViewHost* render_view_host) {
+  // BackgroundContents only loads once, so this can only be the first time it
+  // has started loading.
   FOR_EACH_OBSERVER(extensions::DeferredStartRenderHostObserver,
                     deferred_start_render_host_observer_list_,
-                    OnDeferredStartRenderHostDidStartLoading(this));
+                    OnDeferredStartRenderHostDidStartFirstLoad(this));
 }
 
 void BackgroundContents::DidStopLoading(
     content::RenderViewHost* render_view_host) {
+  // BackgroundContents only loads once, so this can only be the first time
+  // it has stopped loading.
   FOR_EACH_OBSERVER(extensions::DeferredStartRenderHostObserver,
                     deferred_start_render_host_observer_list_,
-                    OnDeferredStartRenderHostDidStopLoading(this));
+                    OnDeferredStartRenderHostDidStopFirstLoad(this));
 }
 
 void BackgroundContents::Observe(int type,
                                  const content::NotificationSource& source,
                                  const content::NotificationDetails& details) {
   // TODO(rafaelw): Implement pagegroup ref-counting so that non-persistent
   // background pages are closed when the last referencing frame is closed.
   switch (type) {
     case chrome::NOTIFICATION_PROFILE_DESTROYED:
     case chrome::NOTIFICATION_APP_TERMINATING: {
@@ skipping 18 matching lines @@
 
 void BackgroundContents::AddDeferredStartRenderHostObserver(
     extensions::DeferredStartRenderHostObserver* observer) {
   deferred_start_render_host_observer_list_.AddObserver(observer);
 }
 
 void BackgroundContents::RemoveDeferredStartRenderHostObserver(
     extensions::DeferredStartRenderHostObserver* observer) {
   deferred_start_render_host_observer_list_.RemoveObserver(observer);
 }