Fix a bug where an extension's icon wasn't being properly reloaded if it changes

chrome/browser/resources/extensions/extension_list.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options', function() {
   'use strict';
 
   /**
    * A lookup helper function to find the first node that has an id (starting
    * at |node| and going up the parent chain).
@@ skipping 17 matching lines @@
   /**
    * @type {Object.<string, boolean>} A map from extension id to a boolean
    *     indicating whether the incognito warning is showing. This persists
    *     between calls to decorate.
    */
   var butterBarVisibility = {};
 
   ExtensionsList.prototype = {
     __proto__: HTMLDivElement.prototype,
 
+    // Appended to icon URL as a query string to prevent icon caching.
+    // Increments each time an extension is reloaded.

[not at google - send to devlin, 2013/02/12 22:57:52]

use jsdoc style

/**
 * @type {number} Appended to icon URL...
 */

[epeterson, 2013/02/12 23:52:42]

Done.

+    iconQueryId_: 0,
+
     /** @override */
     decorate: function() {
       this.textContent = '';
 
       this.showExtensionNodes_();
     },
 
     getIdQueryParam_: function() {
       return parseQueryParams(document.location)['id'];
     },
@@ skipping 42 matching lines @@
         node.classList.add('inactive-extension');
 
       if (!extension.userModifiable)
         node.classList.add('may-not-disable');
 
       var id_to_highlight = this.getIdQueryParam_();
       if (node.id == id_to_highlight)
         node.classList.add('extension-highlight');
 
       var item = node.querySelector('.extension-list-item');
-      item.style.backgroundImage = 'url(' + extension.icon + ')';
+      item.style.backgroundImage = 'url(' + extension.icon + '?' +
+                                   (++this.iconQueryId_) + ')';

[not at google - send to devlin, 2013/02/12 22:57:52]

I think () around ++this.iconQueryId_ are unnecessary, also postincrement is
fine.

[epeterson, 2013/02/12 23:52:42]

Done.

 
       var title = node.querySelector('.extension-title');
       title.textContent = extension.name;
 
       var version = node.querySelector('.extension-version');
       version.textContent = extension.version;
 
       var disableReason = node.querySelector('.extension-disable-reason');
       disableReason.textContent = extension.disableReason;
 
@@ skipping 217 matching lines @@
       }
 
       this.appendChild(node);
     }
   };
 
   return {
     ExtensionsList: ExtensionsList
   };
 });