Support getting offline pages also by original URL

chrome/browser/android/offline_pages/offline_page_utils.h

Index: chrome/browser/android/offline_pages/offline_page_utils.h
diff --git a/chrome/browser/android/offline_pages/offline_page_utils.h b/chrome/browser/android/offline_pages/offline_page_utils.h
index d068b3ca30f5fda86cfb6e06da43c8ab9c6e3e2c..3e74355d8591a1b46dd4ac0d63f42500f71b035f 100644
--- a/chrome/browser/android/offline_pages/offline_page_utils.h
+++ b/chrome/browser/android/offline_pages/offline_page_utils.h
@@ -27,9 +27,10 @@ class OfflinePageUtils {
   // page will be preferred over an older one. The offline page captured from
   // last visit in the tab will not be considered if its tab id does not match
   // the provided |tab_id|.
-  static void SelectPageForOnlineURL(
+  static void SelectPageForURL(
       content::BrowserContext* browser_context,
-      const GURL& online_url,
+      const GURL& url,
+      bool search_by_final_url_only,

[fgorski, 2016/11/16 23:43:46]

Can we have an enum for this, as opposed to a boolean?
I don't like that we put this everywhere in the code:
false /* search_by_final_url_only */

[jianli, 2016/11/17 01:12:14]

Done.

       int tab_id,
       const base::Callback<void(const OfflinePageItem*)>& callback);