Change browser/page action default icon defined in manifest to support hidpi.

chrome/common/extensions/extension_action.cc

 // Copyright (c) 2012 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/common/extensions/extension_action.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "chrome/common/badge_util.h"
+#include "chrome/common/extensions/extension_constants.h"
 #include "googleurl/src/gurl.h"
 #include "grit/theme_resources.h"
 #include "grit/ui_resources.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkDevice.h"
 #include "third_party/skia/include/core/SkPaint.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "ui/base/resource/resource_bundle.h"
@@ skipping 31 matching lines @@
 const int kPadding = 2;
 // The padding between the top of the badge and the top of the text.
 const int kTopTextPadding = -1;
 #endif
 
 const int kBadgeHeight = 11;
 const int kMaxTextWidth = 23;
 // The minimum width for center-aligning the badge.
 const int kCenterAlignThreshold = 20;
 
+void CopyExtensionIconSet(const ExtensionIconSet* from, ExtensionIconSet* to) {

[Jeffrey Yasskin, 2012/09/13 00:23:36]

If we want this function, I suspect ExtensionIconSet should just grow a copy
constructor and assignment operator.

[tbarzic, 2012/09/13 02:01:01]

I don't think we want this functionality in production code (this is used only
in test).

+  for (ExtensionIconSet::IconMap::const_iterator iter = from->map().begin();
+       iter != from->map().end();
+       ++iter) {
+    to->Add(iter->first, iter->second);
+  }
+}
+
+int GetDesiredIconSizeForActionType(ExtensionAction::Type type) {
+  switch (type) {
+    case ExtensionAction::TYPE_BROWSER:
+    case ExtensionAction::TYPE_PAGE:
+      return extension_misc::EXTENSION_ICON_ACTION;
+    case ExtensionAction::TYPE_SCRIPT_BADGE:
+      return extension_misc::EXTENSION_ICON_BITTY;
+    default:
+      NOTREACHED();
+      return 0;
+  }
+}
+
+gfx::ImageSkia GetIconFromIconSet(const ExtensionIconSet* icon_set,

[Jeffrey Yasskin, 2012/09/13 00:23:36]

This function doesn't seem to pull its weight. I'd inline it.

[tbarzic, 2012/09/13 02:01:01]

Done.

+                                 ExtensionIconFactoryDelegate* icon_factory,
+                                 ExtensionAction::Type type) {
+  if (!icon_set || !icon_factory)
+    return gfx::ImageSkia();
+
+  return icon_factory->GetIcon(icon_set, GetDesiredIconSizeForActionType(type));
+}
+
 class GetAttentionImageSource : public gfx::ImageSkiaSource {
  public:
   explicit GetAttentionImageSource(const gfx::ImageSkia& icon)
       : icon_(icon) {}
 
   // gfx::ImageSkiaSource overrides:
   virtual gfx::ImageSkiaRep GetImageForScale(ui::ScaleFactor scale_factor)
       OVERRIDE {
     gfx::ImageSkiaRep icon_rep = icon_.GetRepresentation(scale_factor);
     color_utils::HSL shift = {-1, 0, 0.5};
@@ skipping 173 matching lines @@
       new ExtensionAction(extension_id_, action_type_));
   copy->popup_url_ = popup_url_;
   copy->title_ = title_;
   copy->icon_ = icon_;
   copy->icon_index_ = icon_index_;
   copy->badge_text_ = badge_text_;
   copy->badge_background_color_ = badge_background_color_;
   copy->badge_text_color_ = badge_text_color_;
   copy->appearance_ = appearance_;
   copy->icon_animation_ = icon_animation_;
-  copy->default_icon_path_ = default_icon_path_;
   copy->id_ = id_;
-  copy->icon_paths_ = icon_paths_;
+
+  if (default_icon_.get()) {
+    scoped_ptr<ExtensionIconSet> default_icon(new ExtensionIconSet());
+    CopyExtensionIconSet(default_icon_.get(), default_icon.get());
+    copy->default_icon_ = default_icon.Pass();
+  }
+
+  for (size_t i = 0; i < page_action_icons_.size(); i++) {
+    scoped_ptr<ExtensionIconSet> page_action_icon(new ExtensionIconSet());
+    CopyExtensionIconSet(page_action_icons_[i], page_action_icon.get());
+    copy->AddPageActionIcon(page_action_icon.Pass());
+  }
+
   return copy.Pass();
 }
 
+void ExtensionAction::AddPageActionIcon(scoped_ptr<ExtensionIconSet> icon_set) {
+  page_action_icons_.push_back(icon_set.release());
+}
+
+bool ExtensionAction::IsValidIconIndex(int index) const {
+  return index >= 0 && (static_cast<size_t>(index) < page_action_icons_.size());
+}
+
 void ExtensionAction::SetPopupUrl(int tab_id, const GURL& url) {
   // We store |url| even if it is empty, rather than removing a URL from the
   // map.  If an extension has a default popup, and removes it for a tab via
   // the API, we must remember that there is no popup for that specific tab.
   // If we removed the tab's URL, GetPopupURL would incorrectly return the
   // default URL.
   SetValue(&popup_url_, tab_id, url);
 }
 
 bool ExtensionAction::HasPopup(int tab_id) const {
   return !GetPopupUrl(tab_id).is_empty();
 }
 
 GURL ExtensionAction::GetPopupUrl(int tab_id) const {
   return GetValue(&popup_url_, tab_id);
 }
 
-void ExtensionAction::CacheIcon(const std::string& path,
-                                const gfx::Image& icon) {
-  if (!icon.IsEmpty())
-    path_to_icon_cache_.insert(std::make_pair(path, *icon.ToImageSkia()));
-}
-
 void ExtensionAction::SetIcon(int tab_id, const gfx::Image& image) {
   SetValue(&icon_, tab_id, image.AsImageSkia());
 }
 
-gfx::Image ExtensionAction::GetIcon(int tab_id) const {
+gfx::Image ExtensionAction::GetIcon(
+    int tab_id,
+    ExtensionIconFactoryDelegate* icon_factory) const {
   // Check if a specific icon is set for this tab.
   gfx::ImageSkia icon = GetExplicitlySetIcon(tab_id);
   if (icon.isNull()) {
-    // Need to find an icon from a path.
-    const std::string* path = NULL;
     // Check if one of the elements of icon_path() was selected.
     int icon_index = GetIconIndex(tab_id);
+    const ExtensionIconSet* icon_set = NULL;
     if (icon_index >= 0) {
-      path = &icon_paths()->at(icon_index);
+      icon_set = page_action_icons_[icon_index];
     } else {
-      // Otherwise, use the default icon.
-      path = &default_icon_path();
+      icon_set = default_icon_.get();
     }
 
-    std::map<std::string, gfx::ImageSkia>::const_iterator cached_icon =
-        path_to_icon_cache_.find(*path);
-    if (cached_icon != path_to_icon_cache_.end()) {
-      icon = cached_icon->second;
-    } else {
+    icon = GetIconFromIconSet(icon_set, icon_factory, action_type());
+
+    // Extension favicon is our last resort.
+    if (icon.isNull()) {
       icon = *ui::ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
           IDR_EXTENSIONS_FAVICON);
     }
   }
 
   if (GetValue(&appearance_, tab_id) == WANTS_ATTENTION)
     icon = gfx::ImageSkia(new GetAttentionImageSource(icon), icon.size());
 
   return gfx::Image(ApplyIconAnimation(tab_id, icon));
 }
 
 gfx::ImageSkia ExtensionAction::GetExplicitlySetIcon(int tab_id) const {
   return GetValue(&icon_, tab_id);
 }
 
 void ExtensionAction::SetIconIndex(int tab_id, int index) {
-  if (static_cast<size_t>(index) >= icon_paths_.size()) {
+  if (static_cast<size_t>(index) >= page_action_icons_.size()) {
     NOTREACHED();
     return;
   }
   SetValue(&icon_index_, tab_id, index);
 }
 
 bool ExtensionAction::SetAppearance(int tab_id, Appearance new_appearance) {
   const Appearance old_appearance = GetValue(&appearance_, tab_id);
 
   if (old_appearance == new_appearance)
     return false;
 
+  // Remeber icon width so it can be used for badge painting.

[Jeffrey Yasskin, 2012/09/13 00:23:36]

This doesn't look like the width being rembered.

[tbarzic, 2012/09/13 02:01:01]

Done.

   SetValue(&appearance_, tab_id, new_appearance);
 
   // When showing a badge for the first time on a web page, fade it
   // in.  Other transitions happen instantly.
   if (old_appearance == INVISIBLE && tab_id != kDefaultTabId) {
     RunIconAnimation(tab_id);
   }
 
   return true;
 }
@@ skipping 12 matching lines @@
 
 void ExtensionAction::PaintBadge(gfx::Canvas* canvas,
                                  const gfx::Rect& bounds,
                                  int tab_id) {
   ExtensionAction::DoPaintBadge(
       canvas,
       bounds,
       GetBadgeText(tab_id),
       GetBadgeTextColor(tab_id),
       GetBadgeBackgroundColor(tab_id),
-      GetValue(&icon_, tab_id).size().width());
+      GetIconWidth(tab_id));
 }
 
 gfx::ImageSkia ExtensionAction::GetIconWithBadge(
     const gfx::ImageSkia& icon,
     int tab_id,
     const gfx::Size& spacing) const {
   if (tab_id < 0)
     return icon;
 
   return gfx::ImageSkia(
       new IconWithBadgeImageSource(icon,
                                    spacing,
                                    GetBadgeText(tab_id),
                                    GetBadgeTextColor(tab_id),
                                    GetBadgeBackgroundColor(tab_id)),
      icon.size());
 }
 
+// Determines which icon would be returned by |GetIcon|, and returns its width.
+int ExtensionAction:: GetIconWidth(int tab_id) const {

[Jeffrey Yasskin, 2012/09/13 00:23:36]

No space between :: and the method name.

[tbarzic, 2012/09/13 02:01:01]

Done.

+  // If icon has been set, return its width.
+  gfx::ImageSkia icon = GetValue(&icon_, tab_id);
+  if (!icon.isNull())
+    return icon.width();
+  // If page action icon has been set, or there is a default icon, the icon
+  // the icon width will be set depending on our action type.

[Jeffrey Yasskin, 2012/09/13 00:23:36]

typo: "the icon the icon width"

[tbarzic, 2012/09/13 02:01:01]

Done.

+  int icon_index = GetIconIndex(tab_id);
+  if (icon_index >= 0 || default_icon_.get())
+    return GetDesiredIconSizeForActionType(action_type());
+
+  // If no icon has been set and there is no default icon, we need favicon
+  // width.
+  return ui::ResourceBundle::GetSharedInstance().GetImageNamed(
+          IDR_EXTENSIONS_FAVICON).ToImageSkia()->width();
+}
+
 // static
 void ExtensionAction::DoPaintBadge(gfx::Canvas* canvas,
                                    const gfx::Rect& bounds,
                                    const std::string& text,
                                    const SkColor& text_color_in,
                                    const SkColor& background_color_in,
                                    int icon_width) {
   if (text.empty())
     return;
 
@@ skipping 115 matching lines @@
   icon_animation->Start();
   // After the icon is finished fading in (plus some padding to handle random
   // timer delays), destroy it. We use a delayed task so that the Animation is
   // deleted even if it hasn't finished by the time the MessageLoop is
   // destroyed.
   MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(&DestroyIconAnimation, base::Passed(icon_animation.Pass())),
       base::TimeDelta::FromMilliseconds(kIconFadeInDurationMs * 2));
 }