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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java

Issue 2857333002: [TTS] Write initial Tap-features to Ranker. (Closed)
Patch Set: Added logging of the CS-model URL and updated console-logging output. Created 3 years, 7 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/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java
index 5a000d8ede43c58854518a2a7ece8cb6655c0f7b..a344e805f6610595cb4f8c2bf76211bff748ad6f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchRankerLogger.java
@@ -4,6 +4,8 @@
package org.chromium.chrome.browser.contextualsearch;
+import java.net.URL;
+
/**
* An interface for logging to UMA via Ranker.
*/
@@ -26,6 +28,12 @@ public interface ContextualSearchRankerLogger {
PREVIOUS_28DAY_CTR_PERCENT
}
+ /**
+ * Sets the URL for the page.
+ * @param basePageUrl The URL of the base page to log with Ranker.
+ */
+ void setBasePageUrl(URL basePageUrl);
+
/**
* Logs a particular key/value pair.
* @param feature The feature to log.

Powered by Google App Engine
This is Rietveld 408576698