Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(44)

Unified Diff: extensions/browser/extension_host.h

Issue 1012823002: Revert of Make LoadMonitoringExtensionHostQueue remove itself as an ExtensionHost observer at the... (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/views/extensions/extension_view_views.cc ('k') | extensions/browser/extension_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/extension_host.h
diff --git a/extensions/browser/extension_host.h b/extensions/browser/extension_host.h
index 5d92335da1856cfca170c0ef2158065adac85791..9926161e2904ccf632a79f17f8a4320b9ccb1843 100644
--- a/extensions/browser/extension_host.h
+++ b/extensions/browser/extension_host.h
@@ -59,7 +59,7 @@
content::WebContents* host_contents() const { return host_contents_.get(); }
content::RenderViewHost* render_view_host() const;
content::RenderProcessHost* render_process_host() const;
- bool has_loaded_once() const { return has_loaded_once_; }
+ bool did_stop_loading() const { return did_stop_loading_; }
bool document_element_available() const {
return document_element_available_;
}
@@ -131,8 +131,8 @@
UnloadedExtensionInfo::Reason reason) override;
protected:
- // Called each time this ExtensionHost completes a load finishes loading,
- // before any stop-loading notifications or observer methods are called.
+ // Called after the extension page finishes loading but before the
+ // EXTENSION_HOST_DID_STOP_LOADING notification is sent.
virtual void OnDidStopLoading();
// Navigates to the initial page.
@@ -155,9 +155,6 @@
void OnIncrementLazyKeepaliveCount();
void OnDecrementLazyKeepaliveCount();
- // Records UMA for load events.
- void RecordStopLoadingUMA();
-
// Delegate for functionality that cannot exist in the extensions module.
scoped_ptr<ExtensionHostDelegate> delegate_;
@@ -178,10 +175,8 @@
// host, so we can send messages to it before it finishes loading.
content::RenderViewHost* render_view_host_;
- // Whether the ExtensionHost has finished loading some content at least once.
- // There may be subsequent loads - such as reloads and navigations - and this
- // will not affect its value (it will remain true).
- bool has_loaded_once_;
+ // Whether the RenderWidget has reported that it has stopped loading.
+ bool did_stop_loading_;
// True if the main frame has finished parsing.
bool document_element_available_;
« no previous file with comments | « chrome/browser/ui/views/extensions/extension_view_views.cc ('k') | extensions/browser/extension_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698