Support getting offline pages also by original URL

chrome/browser/android/offline_pages/offline_page_utils.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_
 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_
 
 #include <stdint.h>
 
 #include "base/callback.h"
 
 class GURL;
 
 namespace content {
 class BrowserContext;
 class WebContents;
 }
 
 namespace offline_pages {
 struct OfflinePageHeader;
 struct OfflinePageItem;
 
 class OfflinePageUtils {
  public:
   // Returns via callback an offline page saved for |online_url|, if any. The
   // page is chosen based on creation date; a more recently created offline
   // 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);
 
   // Gets the offline page corresponding to the given web contents.  The
   // returned pointer is owned by the web_contents and may be deleted by user
   // navigation, so it is unsafe to store a copy of the returned pointer.
   static const OfflinePageItem* GetOfflinePageFromWebContents(
       content::WebContents* web_contents);
 
   // Gets the offline header provided when loading the offline page for the
@@ skipping 20 matching lines @@
       const std::string name_space,
       const GURL& offline_page_url,
       const base::Callback<void(bool)>& callback);
 
   static bool EqualsIgnoringFragment(const GURL& lhs, const GURL& rhs);
 };
 
 }  // namespace offline_pages
 
 #endif  // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_