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

Unified Diff: components/offline_pages/content/background_loader/background_loader_contents.h

Issue 2534673002: [Offline pages] Create offliner that uses background loader (Closed)
Patch Set: bleh it works now 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: components/offline_pages/content/background_loader/background_loader_contents.h
diff --git a/components/offline_pages/content/background_loader/background_loader_contents.h b/components/offline_pages/content/background_loader/background_loader_contents.h
index 2432c84d1921aa50ade28a29fe6a1ab21c1b46b9..43b6ceae485b5e68546580b0f388692cf5fd496d 100644
--- a/components/offline_pages/content/background_loader/background_loader_contents.h
+++ b/components/offline_pages/content/background_loader/background_loader_contents.h
@@ -28,9 +28,9 @@ class BackgroundLoaderContents : public content::WebContentsDelegate {
// Loads the URL in a WebContents. Will call observe on all current observers
// with the created WebContents.
- void LoadPage(const GURL& url);
+ virtual void LoadPage(const GURL& url);
// Cancels loading of the current page. Calls Close() on internal WebContents.
- void Cancel();
+ virtual void Cancel();
// Returns the inner web contents.
content::WebContents* web_contents() { return web_contents_.get(); }
@@ -77,6 +77,8 @@ class BackgroundLoaderContents : public content::WebContentsDelegate {
private:
friend class BackgroundLoaderContentsTest;
+ friend class BackgroundLoaderContentsStub;
+
BackgroundLoaderContents();
std::unique_ptr<content::WebContents> web_contents_;

Powered by Google App Engine
This is Rietveld 408576698