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

Unified Diff: chrome/renderer/searchbox/searchbox.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/renderer/searchbox/searchbox.cc
diff --git a/chrome/renderer/searchbox/searchbox.cc b/chrome/renderer/searchbox/searchbox.cc
index 893b3dfd516477207314525dd4f37b3bd60dab57..baf906ba2d3b05d19328deaa0d5e87982239d992 100644
--- a/chrome/renderer/searchbox/searchbox.cc
+++ b/chrome/renderer/searchbox/searchbox.cc
@@ -167,6 +167,12 @@ void SearchBox::LogImpression(int position, const base::string16& provider) {
provider));
}
+void SearchBox::LogNavigation(int position, const base::string16& provider) {
+ render_view()->Send(new ChromeViewHostMsg_LogNavigation(
+ render_view()->GetRoutingID(), render_view()->GetPageId(), position,
+ provider));
+}
+
void SearchBox::CheckIsUserSignedInToChromeAs(const base::string16& identity) {
render_view()->Send(new ChromeViewHostMsg_ChromeIdentityCheck(
render_view()->GetRoutingID(), render_view()->GetPageId(), identity));

Powered by Google App Engine
This is Rietveld 408576698