Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5286)

Unified Diff: chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java

Issue 738693002: Opening new "about:blank" tab if all tabs are closed in tab switcher mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java
diff --git a/chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java b/chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java
index c7e11d39773d075376369910872f0777255586b0..c6c0b9723968cdd76decd56ea9d9dfbd73900d72 100644
--- a/chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java
+++ b/chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionPopup.java
@@ -18,7 +18,7 @@ import android.widget.EditText;
import android.widget.ListPopupWindow;
import android.widget.PopupWindow.OnDismissListener;
-
+import org.chromium.chrome.browser.UrlUtilities;
import org.chromium.chrome.browser.omnibox.AutocompleteController;
import org.chromium.chrome.browser.omnibox.AutocompleteController.OnSuggestionsReceivedListener;
import org.chromium.chrome.browser.omnibox.OmniboxSuggestion;
@@ -68,8 +68,8 @@ public class SuggestionPopup implements OnSuggestionsReceivedListener, TextWatch
}
private void navigateToSuggestion(int position) {
- mToolbar.getCurrentTab().loadUrlWithSanitization(
- mSuggestionArrayAdapter.getItem(position).getUrl());
+ mToolbar.getTabManager().openUrl(UrlUtilities.fixupUrl(
+ mSuggestionArrayAdapter.getItem(position).getUrl()));
mUrlField.clearFocus();
mToolbar.setKeyboardVisibilityForUrl(false);
mToolbar.getCurrentTab().getView().requestFocus();

Powered by Google App Engine
This is Rietveld 408576698