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

Unified Diff: chrome/browser/autocomplete/history_url_provider.cc

Issue 3152029: Ctrl-based actions should take precedence over ctrl-enter (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: Cleaning things up Created 10 years, 4 months 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/browser/autocomplete/history_url_provider.cc
diff --git a/chrome/browser/autocomplete/history_url_provider.cc b/chrome/browser/autocomplete/history_url_provider.cc
index b0605550afb1b684a1b4da7b12c65822cd3837bd..9d4ee13c0387a9db4bcce27297c71244b6a9f837 100644
--- a/chrome/browser/autocomplete/history_url_provider.cc
+++ b/chrome/browser/autocomplete/history_url_provider.cc
@@ -403,8 +403,12 @@ bool HistoryURLProvider::FixupExactSuggestion(history::URLDatabase* db,
return false;
GURL destination_url(URLFixerUpper::FixupURL(WideToUTF8(input.text()),
std::string()));
- if (!db->GetRowForURL(destination_url, &info))
+ if (!db->GetRowForURL(destination_url, NULL))
return false;
+
+ // If we got here, then we hit the tricky corner case. Make sure that
+ // |info| corresponds to the right URL.
+ info = history::URLRow(match->destination_url);
} else {
// We have data for this match, use it.
match->deletable = true;

Powered by Google App Engine
This is Rietveld 408576698