Make some updates to extension iconography.

extensions/browser/image_loader.h

Index: extensions/browser/image_loader.h
diff --git a/extensions/browser/image_loader.h b/extensions/browser/image_loader.h
index 3a7c1b4c607024d1b5ece976d77522758fc49538..cf3c407264a6e7a6aed57a4c08a6c55fb0101365 100644
--- a/extensions/browser/image_loader.h
+++ b/extensions/browser/image_loader.h
@@ -80,11 +80,20 @@ class ImageLoader : public KeyedService {
   // if the image was found in the cache.
   // Note this method loads a raw bitmap from the resource. All sizes given are
   // assumed to be in pixels.
+  // TODO(estade): remove this in favor of LoadImageAtEveryScaleFactorAsync,
+  // and rename the later to LoadImageAsync.

[Devlin, 2016/12/15 17:14:41]

s/later/latter

[Evan Stade, 2016/12/15 23:49:02]

Done.

   void LoadImageAsync(const extensions::Extension* extension,
                       const ExtensionResource& resource,
                       const gfx::Size& max_size,
                       const ImageLoaderImageCallback& callback);
 
+  // Loads and returns a gfx::Image that has representations at all supported
+  // scale factors.
+  void LoadImageAtEveryScaleFactorAsync(
+      const extensions::Extension* extension,

[Devlin, 2016/12/15 17:14:41]

no need for extensions:: prefix.

[Evan Stade, 2016/12/15 23:49:02]

Done.

+      const gfx::Size& dip_size,
+      const ImageLoaderImageCallback& callback);
+
   // Same as LoadImageAsync() above except it loads multiple images from the
   // same extension. This is used to load multiple resolutions of the same image
   // type.