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

Unified Diff: chrome/browser/prerender/prerender_contents.h

Issue 2575523002: Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch (Closed)
Patch Set: remove debugging code Created 4 years 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
Index: chrome/browser/prerender/prerender_contents.h
diff --git a/chrome/browser/prerender/prerender_contents.h b/chrome/browser/prerender/prerender_contents.h
index e8acddd8e5e45217aa5b1155e93c1ce8639157f7..d185ee12c8784748bbe9c3ebb552e1d64c62661c 100644
--- a/chrome/browser/prerender/prerender_contents.h
+++ b/chrome/browser/prerender/prerender_contents.h
@@ -298,6 +298,10 @@ class PrerenderContents : public content::NotificationObserver,
// Message handlers.
void OnCancelPrerenderForPrinting();
+ // Actually start prerendering after checking for service workers. Should be
+ // called only as part of StartPrerendering();
+ void ContinuePrerenderIfNoServiceWorker(bool has_service_worker);
+
base::ObserverList<Observer> observer_list_;
// The prerender manager owning this object.
@@ -366,6 +370,8 @@ class PrerenderContents : public content::NotificationObserver,
// transferred over the network for resources. Updated with AddNetworkBytes.
int64_t network_bytes_;
+ base::WeakPtrFactory<PrerenderContents> weak_factory_;
+
DISALLOW_COPY_AND_ASSIGN(PrerenderContents);
};
« no previous file with comments | « no previous file | chrome/browser/prerender/prerender_contents.cc » ('j') | chrome/browser/prerender/prerender_contents.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698