[Offline Pages] Fixing bookmarking offline page will save file path.

chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java

 // Copyright 2014 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.
 
 package org.chromium.chrome.browser.tab;
 
 import android.app.Activity;
 import android.app.Application;
 import android.content.Context;
 import android.content.Intent;
@@ skipping 2880 matching lines @@
      * Shows the list of offline pages. This should only be hit when offline pages feature is
      * enabled.
      */
     @CalledByNative
     public void showOfflinePages() {
         // The offline pages filter view will be loaded by default when offline.
         BookmarkUtils.showBookmarkManager(getActivity());
     }
 
     /**
+     * @return Original url of the tab, if tab currently displays offline copy, would return url of
+     * the original page. Otherwise return the original url from DOMDistiller.
+     */
+    public String getOriginalUrl() {

[Ian Wen, 2016/07/16 01:04:54]

We should convert some other places that we use getOfflinePageOriginalUrl() to
useing this method instead. For example, ToolbarModelImpl.

[romax, 2016/07/18 22:38:09]

Done.

+        if (isOfflinePage()) {
+            return getOfflinePageOriginalUrl();
+        } else {
+            return DomDistillerUrlUtils.getOriginalUrlFromDistillerUrl(getUrl());
+        }
+    }
+
+    /**
      * If a Lo-Fi snackbar has not been shown yet for this page load, a Lo-Fi snackbar is shown.
      *
      * @param isPreview Whether the Lo-Fi response was a preview response.
      */
     @CalledByNative
     public void onLoFiResponseReceived(boolean isPreview) {
         if (getLoFiBarController() != null) {
             mLoFiBarController.maybeCreateLoFiBar(this, isPreview);
         }
     }
@@ skipping 349 matching lines @@
     private native void nativeSetInterceptNavigationDelegate(long nativeTabAndroid,
             InterceptNavigationDelegate delegate);
     private native void nativeAttachToTabContentManager(long nativeTabAndroid,
             TabContentManager tabContentManager);
     private native void nativeAttachOverlayContentViewCore(long nativeTabAndroid,
             ContentViewCore content, boolean visible);
     private native void nativeDetachOverlayContentViewCore(long nativeTabAndroid,
             ContentViewCore content);
     private native boolean nativeHasPrerenderedUrl(long nativeTabAndroid, String url);
 }