First fix to minimize copying of image data.

app/theme_provider.h

 // Copyright (c) 2009 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 APP_THEME_PROVIDER_H_
 #define APP_THEME_PROVIDER_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "third_party/skia/include/core/SkColor.h"
 
 #if defined(OS_LINUX)
 #include <gdk/gdk.h>
 #elif defined(OS_MACOSX)
 #ifdef __OBJC__
 @class NSColor;
 @class NSImage;
 #else
 class NSColor;
 class NSImage;
 #endif  // __OBJC__
 #endif  // OS_*
 
 class Profile;
+class RefCountedMemory;
 class SkBitmap;
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // ThemeProvider
 //
 //   ThemeProvider is an abstract class that defines the API that should be
 //   implemented to provide bitmaps and color information for a given theme.
 //
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 17 matching lines @@
   virtual bool GetDisplayProperty(int id, int* result) const = 0;
 
   // Whether we should use the native system frame (typically Aero glass) or
   // a custom frame.
   virtual bool ShouldUseNativeFrame() const = 0;
 
   // Whether or not we have a certain image. Used for when the default theme
   // doesn't provide a certain image, but custom themes might (badges, etc).
   virtual bool HasCustomImage(int id) const = 0;
 
-  // Reads the image data from the theme file into the specified vector. Returns
-  // true on success.
-  virtual bool GetRawData(int id,
-                          std::vector<unsigned char>* raw_data) const = 0;
+  // Reads the image data from the theme file into the specified
+  // vector. Returns NULL on error.
+  virtual RefCountedMemory* GetRawData(int id) const = 0;
 
 #if defined(OS_LINUX) && !defined(TOOLKIT_VIEWS)
   // Gets the GdkPixbuf with the specified |id|.  Returns a pointer to a shared
   // instance of the GdkPixbuf.  This shared GdkPixbuf is owned by the theme
   // provider and should not be freed.
   //
   // The bitmap is assumed to exist. This function will log in release, and
   // assert in debug mode if it does not. On failure, this will return a
   // pointer to a shared empty placeholder bitmap so it will be visible what
   // is missing.
@@ skipping 16 matching lines @@
 
   // Gets the NSColor for tinting with the specified |id|.
   //
   // The tint is not assumed to exist. If a theme does not provide a tint with
   // that id, this function will return nil.
   virtual NSColor* GetNSColorTint(int id) const = 0;
 #endif
 };
 
 #endif  // APP_THEME_PROVIDER_H_