Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java |
| index 3e96ac6c9d30ea08b4e941c6c39969ba9d5b838d..4c0a8ef49c2e795bbbed946e9a5f63db9aa38dc4 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java |
| @@ -2064,16 +2064,10 @@ public class LocationBarLayout extends FrameLayout implements OnClickListener, |
| updateCustomSelectionActionModeCallback(); |
| } |
| - /** |
| - * Gets the URL of the web page in the tab. When displaying offline page it gets the URL of the |
| - * original page. |
| - */ |
| + /** Gets the URL of the web page in the tab. */ |
| private String getOnlineUrlFromTab() { |
|
Dmitry Titov
2016/08/20 01:08:13
I wonder if this method deserves to exist, it's pr
jianli
2016/08/20 01:48:52
I think it is needed since tab might be null. Rena
|
| Tab currentTab = getCurrentTab(); |
| if (currentTab == null) return ""; |
| - if (currentTab.isOfflinePage()) { |
| - return currentTab.getOriginalUrl().trim(); |
| - } |
| return currentTab.getUrl().trim(); |
| } |