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

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

Issue 2044613003: Adds unit tests for offline_pages::PrerenderAdapter calling PrerenderManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated DCHECK to ASSERT Created 4 years, 6 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/prerender/prerender_browsertest.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_manager.h
diff --git a/chrome/browser/prerender/prerender_manager.h b/chrome/browser/prerender/prerender_manager.h
index 9ed133e1e47da0f834547c3440cdc62306aa096c..e7b27e114e9080043a3c1975f1879fc4f394f140 100644
--- a/chrome/browser/prerender/prerender_manager.h
+++ b/chrome/browser/prerender/prerender_manager.h
@@ -55,6 +55,10 @@ namespace gfx {
class Size;
}
+namespace offline_pages {
+class PrerenderAdapterTest;
+}
+
namespace prerender {
class PrerenderHandle;
@@ -319,6 +323,9 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
// content::RenderProcessHostObserver implementation.
void RenderProcessHostDestroyed(content::RenderProcessHost* host) override;
+ void SetPrerenderContentsFactoryForTest(
+ PrerenderContents::Factory* prerender_contents_factory);
+
protected:
class PrerenderData : public base::SupportsWeakPtr<PrerenderData> {
public:
@@ -378,9 +385,6 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
DISALLOW_COPY_AND_ASSIGN(PrerenderData);
};
- void SetPrerenderContentsFactory(
- PrerenderContents::Factory* prerender_contents_factory);
-
// Called by a PrerenderData to signal that the launcher has navigated away
// from the context that launched the prerender. A user may have clicked
// a link in a page containing a <link rel=prerender> element, or the user
@@ -389,7 +393,6 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
void SourceNavigatedAway(PrerenderData* prerender_data);
private:
- friend class ::InstantSearchPrerendererTest;
friend class PrerenderBrowserTest;
friend class PrerenderContents;
friend class PrerenderHandle;
« no previous file with comments | « chrome/browser/prerender/prerender_browsertest.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698