Eliminate all UI thread decoding of extension images....

chrome/browser/cocoa/location_bar_view_mac.h

 // Copyright (c) 2010 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.
 
 #ifndef CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_
 #define CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_
 
 #include <string>
 #include <map>
 #include <vector>
@@ skipping 223 matching lines @@
     // When a new page action is created, all the icons are destroyed and
     // recreated; at this point we need to calculate sizes to lay out the
     // icons even though no images are available yet.  For this case, we return
     // the default image size for a page icon.
     virtual NSSize GetPreferredImageSize();
 
     // Either notify listeners or show a popup depending on the Page Action.
     virtual void OnMousePressed(NSRect bounds);
 
     // Overridden from ImageLoadingTracker.
-    virtual void OnImageLoaded(SkBitmap* image, size_t index);
+    virtual void OnImageLoaded(
+        SkBitmap* image, ExtensionResource resource, int index);
 
     // Called to notify the Page Action that it should determine whether to be
     // visible or hidden. |contents| is the TabContents that is active, |url|
     // is the current page URL.
     void UpdateVisibility(TabContents* contents, const GURL& url);
 
     // Sets the tooltip for this Page Action image.
     void SetToolTip(NSString* tooltip);
     void SetToolTip(std::string tooltip);
 
     // Returns the tooltip for this Page Action image or |nil| if there is none.
     virtual const NSString* GetToolTip();
 
     // Overridden to return a menu.
     virtual NSMenu* GetMenu();
 
    protected:
     // For unit testing only.
     PageActionImageView() : owner_(NULL),
                             profile_(NULL),
                             page_action_(NULL),
-                            tracker_(NULL),
+                            tracker_(this),
                             current_tab_id_(-1),
                             preview_enabled_(false) {}
 
    private:
     // Overridden from NotificationObserver:
     virtual void Observe(NotificationType type,
                          const NotificationSource& source,
                          const NotificationDetails& details);
 
     // The location bar view that owns us.
     LocationBarViewMac* owner_;
 
     // The current profile (not owned by us).
     Profile* profile_;
 
     // The Page Action that this view represents. The Page Action is not owned
     // by us, it resides in the extension of this particular profile.
     ExtensionAction* page_action_;
 
     // A cache of images the Page Actions might need to show, mapped by path.
     typedef std::map<std::string, SkBitmap> PageActionMap;
     PageActionMap page_action_icons_;
 
     // The object that is waiting for the image loading to complete
     // asynchronously.
-    ImageLoadingTracker* tracker_;
+    ImageLoadingTracker tracker_;
 
     // The tab id we are currently showing the icon for.
     int current_tab_id_;
 
     // The URL we are currently showing the icon for.
     GURL current_url_;
 
     // The string to show for a tooltip.
     scoped_nsobject<NSString> tooltip_;
 
@@ skipping 125 matching lines @@
   // The transition type to use for the navigation.
   PageTransition::Type transition_;
 
   // Used to register for notifications received by NotificationObserver.
   NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(LocationBarViewMac);
 };
 
 #endif  // CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_