NTP5: Refactoring appData to use Tile's data implementation.

chrome/browser/resources/ntp_search/thumbnail_page.js

Index: chrome/browser/resources/ntp_search/thumbnail_page.js
diff --git a/chrome/browser/resources/ntp_search/thumbnail_page.js b/chrome/browser/resources/ntp_search/thumbnail_page.js
index 7a438d674ee672a26c3fef7176bdab85b13ba9e7..77ab17fccc5fcb3b2a7d49e6f63373bcacb159b5 100644
--- a/chrome/browser/resources/ntp_search/thumbnail_page.js
+++ b/chrome/browser/resources/ntp_search/thumbnail_page.js
@@ -56,8 +56,9 @@ cr.define('ntp', function() {
      * Update the appearance of this tile according to |data|.
      * @param {Object} data A dictionary of relevant data for the page.
      */
-    setData: function(data) {
-      Tile.prototype.setData.apply(this, arguments);
+    set data(data) {
+      Object.getOwnPropertyDescriptor(Tile.prototype, 'data').set.apply(this,
+          arguments);
 
       this.formatThumbnail_(data);
     },

[Dan Beam, 2012/12/05 18:34:05]

I'm not totally sure, but you might need a `get data` as well - as long as it
works for you...

[pedro (no code reviews), 2012/12/05 19:41:34]

Thumbnail page is basically a virtual class, and will never be instantiated,
that's why I did not include the getter here. The subclass (MostVisisted) has
the getter though.