First fix to minimize copying of image data.

chrome/browser/dom_ui/dom_ui_factory.cc

 // 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.
 
 #include "chrome/browser/dom_ui/dom_ui_factory.h"
 
 #include "chrome/browser/dom_ui/downloads_ui.h"
 #include "chrome/browser/dom_ui/devtools_ui.h"
 #include "chrome/browser/dom_ui/history_ui.h"
 #include "chrome/browser/dom_ui/html_dialog_ui.h"
@@ skipping 102 matching lines @@
 // static
 DOMUI* DOMUIFactory::CreateDOMUIForURL(TabContents* tab_contents,
                                        const GURL& url) {
   DOMUIFactoryFunction function = GetDOMUIFactoryFunction(url);
   if (!function)
     return NULL;
   return (*function)(tab_contents, url);
 }
 
 // static
-bool DOMUIFactory::GetFaviconResourceBytes(const GURL& page_url,
-                                           std::vector<unsigned char>* bytes) {
+RefCountedMemory* DOMUIFactory::GetFaviconResourceBytes(
+    const GURL& page_url) {
   if (!HasDOMUIScheme(page_url))
-    return false;
+    return NULL;
 
   if (page_url.host() == chrome::kChromeUIHistoryHost)
-    return HistoryUI::GetFaviconResourceBytes(bytes);
+    return HistoryUI::GetFaviconResourceBytes();
 
   if (page_url.host() == chrome::kChromeUIDownloadsHost)
-    return DownloadsUI::GetFaviconResourceBytes(bytes);
+    return DownloadsUI::GetFaviconResourceBytes();
 
-  return false;
+  return NULL;
 }