Fixes for turning on offliner request.

chrome/browser/android/offline_pages/prerendering_loader.h

Index: chrome/browser/android/offline_pages/prerendering_loader.h
diff --git a/chrome/browser/android/offline_pages/prerendering_loader.h b/chrome/browser/android/offline_pages/prerendering_loader.h
index 22659e96a3609d311ef21505beda494329244112..3e27340b66c590d1a2ad0db4f2a75bc09b556109 100644
--- a/chrome/browser/android/offline_pages/prerendering_loader.h
+++ b/chrome/browser/android/offline_pages/prerendering_loader.h
@@ -46,7 +46,7 @@ class PrerenderingLoader : public PrerenderAdapter::Observer {
   // once - first for a successful load and then if canceled after the
   // load (which may be from resources being reclaimed) at which point
   // the retrieved WebContents should no longer be used.
-  virtual bool LoadPage(const GURL& url, const LoadPageCallback& callback);
+  virtual bool LoadPage(const GURL& url, LoadPageCallback callback);

[Pete Williamson, 2016/06/08 17:32:16]

Generally speaking, we like to pass classes by pointer or reference unless they
are small (std::String is the typical example of a small class, with only one
pointer member).  I can understand if your code cannot hold onto a reference,
but would a pointer work?  If so, be sure to use unique_ptr if transferring
ownership, or a bald pointer if ownership is managed elsewhere (and document
that it is unowned).

[dougarnett, 2016/06/08 20:02:31]

Done - holding copy of callback, fixed param signature

 
   // Stops (completes or cancels) the load request. Must be called when
   // LoadPageCallback is done with consuming the contents. May be called