Commit 49a603c1 authored by John Reck's avatar John Reck
Browse files

Use OriginalUrl instead of Url for history

 Bug: 3500263

Change-Id: I8a0b795dcac28e016e8511aa856aa3e9a45d6a13
parent 02f97585
...@@ -858,7 +858,7 @@ public class Controller ...@@ -858,7 +858,7 @@ public class Controller
@Override @Override
public void onReceivedTitle(Tab tab, final String title) { public void onReceivedTitle(Tab tab, final String title) {
mUi.onTabDataChanged(tab); mUi.onTabDataChanged(tab);
final String pageUrl = tab.getUrl(); final String pageUrl = tab.getOriginalUrl();
if (TextUtils.isEmpty(pageUrl) || pageUrl.length() if (TextUtils.isEmpty(pageUrl) || pageUrl.length()
>= SQLiteDatabase.SQLITE_MAX_LIKE_PATTERN_LENGTH) { >= SQLiteDatabase.SQLITE_MAX_LIKE_PATTERN_LENGTH) {
return; return;
...@@ -906,7 +906,7 @@ public class Controller ...@@ -906,7 +906,7 @@ public class Controller
public void doUpdateVisitedHistory(Tab tab, boolean isReload) { public void doUpdateVisitedHistory(Tab tab, boolean isReload) {
// Don't save anything in private browsing mode // Don't save anything in private browsing mode
if (tab.isPrivateBrowsingEnabled()) return; if (tab.isPrivateBrowsingEnabled()) return;
String url = tab.getUrl(); String url = tab.getOriginalUrl();
if (TextUtils.isEmpty(url) if (TextUtils.isEmpty(url)
|| url.regionMatches(true, 0, "about:", 0, 6)) { || url.regionMatches(true, 0, "about:", 0, 6)) {
......
...@@ -1583,6 +1583,13 @@ class Tab { ...@@ -1583,6 +1583,13 @@ class Tab {
return UrlUtils.filteredUrl(mCurrentState.mUrl); return UrlUtils.filteredUrl(mCurrentState.mUrl);
} }
String getOriginalUrl() {
if (mMainView == null) {
return "";
}
return UrlUtils.filteredUrl(mMainView.getOriginalUrl());
}
/** /**
* Get the title of this tab. * Get the title of this tab.
*/ */
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment