Support getting offline pages also by original URL

chrome/browser/android/offline_pages/offline_page_bridge.cc

 // 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.
 
 #include "chrome/browser/android/offline_pages/offline_page_bridge.h"
 
 #include <memory>
 #include <set>
 #include <string>
 #include <utility>
@@ skipping 390 matching lines @@
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jstring>& j_online_url,
     int tab_id,
     const JavaParamRef<jobject>& j_callback_obj) {
   DCHECK(j_callback_obj);
 
   ScopedJavaGlobalRef<jobject> j_callback_ref;
   j_callback_ref.Reset(env, j_callback_obj);
 
-  OfflinePageUtils::SelectPageForOnlineURL(
+  OfflinePageUtils::SelectPageForURL(
       browser_context_,
       GURL(ConvertJavaStringToUTF8(env, j_online_url)),
+      OfflinePageModel::URLSearchMode::SEARCH_BY_FINAL_URL_ONLY,

[vitaliii, 2016/11/17 18:41:11]

If you replace this with SEARCH_BY_ALL_URLS, then our (NTP's) code will work as
expected with sites, which redirect.
If you don't, we will *not* be terribly broken and no crashes.
Please ask fgorski@ for more details, in case I am not available. 

[jianli, 2016/11/17 20:59:25]

Done.

       tab_id,
       base::Bind(&SingleOfflinePageItemCallback, j_callback_ref));
 }
 
 void OfflinePageBridge::SavePage(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jobject>& j_callback_obj,
     const JavaParamRef<jobject>& j_web_contents,
     const JavaParamRef<jstring>& j_namespace,
@@ skipping 143 matching lines @@
       env, ConvertUTF8ToJavaString(env, client_id.name_space),
       ConvertUTF8ToJavaString(env, client_id.id));
 }
 
 bool RegisterOfflinePageBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace offline_pages