Change panels to use gfx::Image instead of SkBitmap for favicon.

chrome/browser/ui/panels/old_panel.cc

Index: chrome/browser/ui/panels/old_panel.cc
diff --git a/chrome/browser/ui/panels/old_panel.cc b/chrome/browser/ui/panels/old_panel.cc
index 53e9db813b1d92216e1355eed91a68585f1ae10e..a488cdd68e07c5546be272b775518fa84a6d136a 100644
--- a/chrome/browser/ui/panels/old_panel.cc
+++ b/chrome/browser/ui/panels/old_panel.cc
@@ -4,12 +4,15 @@
 
 #include "chrome/browser/ui/panels/old_panel.h"
 
+#include "chrome/browser/favicon/favicon_tab_helper.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_command_controller.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/panels/panel_browser_window.h"
-#include "third_party/skia/include/core/SkBitmap.h"
+#include "chrome/browser/ui/browser_tabstrip.h"
+#include "chrome/browser/ui/tab_contents/tab_contents.h"
+#include "ui/gfx/image/image.h"
 
 OldPanel::OldPanel(Browser* browser,
                    const gfx::Size& min_size, const gfx::Size& max_size)
@@ -58,6 +61,9 @@ void OldPanel::ExecuteCommandWithDisposition(
   chrome::ExecuteCommandWithDisposition(browser_, id, disposition);
 }
 
-SkBitmap OldPanel::GetCurrentPageIcon() const {
-  return browser_->GetCurrentPageIcon();
+gfx::Image OldPanel::GetCurrentPageIcon() const {
+  // Browser has not been changed to return SkBitmap yet so we get the favicon
+  // directly rather than use browser_->GetCurrentPageIcon().

[Nico, 2012/08/10 18:17:33]

Add a TODO to undo this once Browser::GetCurrentPageIcon() is converted

[jennb, 2012/08/11 04:20:05]

No need. This file will be deleted soon after the panels refactor is complete. I
only made this edit to get things compiling in the meanwhile.

+  TabContents* contents = chrome::GetActiveTabContents(browser_);
+  return contents ? contents->favicon_tab_helper()->GetFavicon() : gfx::Image();
 }