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

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

Issue 2117373002: Cleanup: Change LogMostVisitedImpression|Navigation APIs to take an enum (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ntp_uma_cleanup
Patch Set: rebase Created 4 years, 5 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/resources/local_ntp/most_visited_thumbnail.js
diff --git a/chrome/browser/resources/local_ntp/most_visited_thumbnail.js b/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
index 33baada73dabe51b3cea73dbf1f67d492a4de9b4..bf74b6ee6b2a8a46925c0b9b870fb8418b5c6e2e 100644
--- a/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
+++ b/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
@@ -14,9 +14,9 @@ window.addEventListener('DOMContentLoaded', function() {
function logEvent(eventName) {
chrome.embeddedSearch.newTabPage.logEvent(eventName);
}
- function logMostVisitedImpression(tileIndex, provider) {
- chrome.embeddedSearch.newTabPage.logMostVisitedImpression(
- tileIndex, provider);
+ function logMostVisitedImpression(tileIndex, tileSource) {
+ chrome.embeddedSearch.newTabPage.logMostVisitedImpression(tileIndex,
+ tileSource);
}
function displayLink(link) {
document.body.appendChild(link);
@@ -25,7 +25,7 @@ window.addEventListener('DOMContentLoaded', function() {
function showDomainElement() {
var link = createMostVisitedLink(
params, data.url, data.title, undefined, data.direction,
- data.provider);
+ data.tileSource);
var domain = document.createElement('div');
domain.textContent = data.domain;
link.appendChild(domain);
@@ -36,7 +36,7 @@ window.addEventListener('DOMContentLoaded', function() {
function showEmptyTile() {
displayLink(createMostVisitedLink(
params, data.url, data.title, undefined, data.direction,
- data.provider));
+ data.tileSource));
}
// Creates and adds an image.
function createThumbnail(src, imageClass) {
@@ -47,7 +47,7 @@ window.addEventListener('DOMContentLoaded', function() {
image.onload = function() {
var link = createMostVisitedLink(
params, data.url, data.title, undefined, data.direction,
- data.provider);
+ data.tileSource);
// Use blocker to prevent context menu from showing image-related items.
var blocker = document.createElement('span');
blocker.className = 'blocker';
@@ -91,8 +91,8 @@ window.addEventListener('DOMContentLoaded', function() {
logEvent(NTP_LOGGING_EVENT_TYPE.NTP_TILE);
// Log an impression if we know the position of the tile.
- if (isFinite(params.pos) && data.provider) {
- logMostVisitedImpression(parseInt(params.pos, 10), data.provider);
+ if (isFinite(params.pos)) {
+ logMostVisitedImpression(parseInt(params.pos, 10), data.tileSource);
}
});
});
« no previous file with comments | « chrome/browser/resources/local_ntp/most_visited_single.js ('k') | chrome/browser/resources/local_ntp/most_visited_title.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698