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

Unified Diff: chrome/browser/resources/ntp4/most_visited_page.js

Issue 9320051: Fix JavaScript style in ntp4 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 8 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/resources/ntp4/nav_dot.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp4/most_visited_page.js
diff --git a/chrome/browser/resources/ntp4/most_visited_page.js b/chrome/browser/resources/ntp4/most_visited_page.js
index 9d4d8b6f42f675403d8d2cff1290c9b4d610060a..fb3719fcd17b8e2813a5f0a5376279ecbe42ab04 100644
--- a/chrome/browser/resources/ntp4/most_visited_page.js
+++ b/chrome/browser/resources/ntp4/most_visited_page.js
@@ -168,7 +168,7 @@ cr.define('ntp4', function() {
showUndoNotification_: function() {
var data = this.data_;
var self = this;
- var doUndo = function () {
+ var doUndo = function() {
chrome.send('removeURLsFromMostVisitedBlacklist', [data.url]);
self.updateForData(data);
}
@@ -176,14 +176,14 @@ cr.define('ntp4', function() {
var undo = {
action: doUndo,
text: templateData.undothumbnailremove,
- }
+ };
var undoAll = {
action: function() {
chrome.send('clearMostVisitedURLsBlacklist', []);
},
text: templateData.restoreThumbnailsShort,
- }
+ };
ntp4.showNotification(templateData.thumbnailremovednotification,
[undo, undoAll]);
@@ -351,9 +351,10 @@ cr.define('ntp4', function() {
* We've gotten additional Most Visited data. Update our old data with the
* new data. The ordering of the new data is not important, except when a
* page is pinned. Thus we try to minimize re-ordering.
- * @param {Object} oldData The current Most Visited page list.
- * @param {Object} newData The new Most Visited page list.
- * @return The merged page list that should replace the current page list.
+ * @param {Array} oldData The current Most Visited page list.
+ * @param {Array} newData The new Most Visited page list.
+ * @return {Array} The merged page list that should replace the current page
+ * list.
*/
function refreshData(oldData, newData) {
oldData = oldData.slice(0, THUMBNAIL_COUNT);
« no previous file with comments | « no previous file | chrome/browser/resources/ntp4/nav_dot.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698