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

Unified Diff: chrome/browser/ui/search/search_tab_helper.cc

Issue 178253008: Redoing Issue 36073011: Allowing file:/// in Instant Extended's Most Visited links. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adding and using logNavigation(); updating tests; removing ping and the log.html page. Created 6 years, 10 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/ui/search/search_tab_helper.cc
diff --git a/chrome/browser/ui/search/search_tab_helper.cc b/chrome/browser/ui/search/search_tab_helper.cc
index ebfcf5205438decccbf657cdd878c43a97734a54..8f22cc3b2ec422bba54876b88b8e8eb7559163d2 100644
--- a/chrome/browser/ui/search/search_tab_helper.cc
+++ b/chrome/browser/ui/search/search_tab_helper.cc
@@ -544,6 +544,12 @@ void SearchTabHelper::OnLogImpression(int position,
web_contents())->LogImpression(position, provider);
}
+void SearchTabHelper::OnLogNavigation(int position,
+ const base::string16& provider) {
+ NTPUserDataLogger::GetOrCreateFromWebContents(
+ web_contents())->LogNavigation(position, provider);
+}
+
void SearchTabHelper::PasteIntoOmnibox(const base::string16& text) {
OmniboxView* omnibox = GetOmniboxView(web_contents());
if (!omnibox)

Powered by Google App Engine
This is Rietveld 408576698