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

Unified Diff: chrome/browser/resources/local_ntp/most_visited_single.js

Issue 2535803002: NTP: Remove impression/click pings (Closed)
Patch Set: Created 4 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/local_ntp/most_visited_single.js
diff --git a/chrome/browser/resources/local_ntp/most_visited_single.js b/chrome/browser/resources/local_ntp/most_visited_single.js
index 45db544b506c8f7b4c3c1909eab0df4d564f2043..cfd0ab9f79ff988a9017c7705d9858edd25e0594 100644
--- a/chrome/browser/resources/local_ntp/most_visited_single.js
+++ b/chrome/browser/resources/local_ntp/most_visited_single.js
@@ -83,11 +83,6 @@ var tiles = null;
*/
var queryArgs = {};
-/**
- * Url to ping when suggestions have been shown.
- */
-var impressionUrl = null;
-
/**
* Log an event on the NTP.
@@ -279,11 +274,6 @@ var showTiles = function() {
// Make sure the tiles variable contain the next tileset we may use.
tiles = document.createElement('div');
-
- if (impressionUrl) {
- navigator.sendBeacon(impressionUrl);
- impressionUrl = null;
- }
};
@@ -383,14 +373,6 @@ var renderTile = function(data) {
}
tile.setAttribute('aria-label', data.title);
tile.title = data.title;
- if (data.impressionUrl) {
- impressionUrl = data.impressionUrl;
- }
- if (data.pingUrl) {
- tile.addEventListener('click', function(ev) {
- navigator.sendBeacon(data.pingUrl);
- });
- }
tile.addEventListener('click', function(ev) {
logMostVisitedNavigation(position, data.tileSource);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698