[ntp4] Bookmarks page implementation, first-pass.

chrome/browser/resources/ntp4/most_visited_page.js

Index: chrome/browser/resources/ntp4/most_visited_page.js
===================================================================
--- chrome/browser/resources/ntp4/most_visited_page.js	(revision 96479)
+++ chrome/browser/resources/ntp4/most_visited_page.js	(working copy)
@@ -84,7 +84,7 @@
       }
 
       var id = tileID++;
-      this.setAttribute('id', 'tile' + id);
+      this.id = 'most_visited_tile_' + id;

[arv (Not doing code reviews), 2011/08/15 17:46:30]

most-visited-tile

[csilv, 2011/08/15 18:24:28]

Done.

       this.data_ = data;
       // TODO(estade): this shouldn't be focusable if the page isn't showing.
       this.tabIndex = 0;
@@ -97,7 +97,8 @@
       if (data.faviconDominantColor)
         this.setStripeColor(data.faviconDominantColor);
       else
-        chrome.send('getFaviconDominantColor', [faviconUrl, id]);
+        chrome.send('getFaviconDominantColor',
+            [faviconUrl, id, 'ntp4.setMostVisitedFaviconDominantColor']);
 
       var title = this.querySelector('.title');
       title.textContent = data.title;
@@ -427,8 +428,8 @@
     return oldData;
   };
 
-  function setFaviconDominantColor(id, color) {
-    var tile = $('tile' + id);
+  function setMostVisitedFaviconDominantColor(id, color) {
+    var tile = $('most_visited_tile_' + id);
     if (tile)
       tile.setStripeColor(color);
   };
@@ -436,6 +437,6 @@
   return {
     MostVisitedPage: MostVisitedPage,
     refreshData: refreshData,
-    setFaviconDominantColor: setFaviconDominantColor,
+    setMostVisitedFaviconDominantColor: setMostVisitedFaviconDominantColor,
   };
 });