Refresh for the Chrome notifications image template.

ui/message_center/views/notification_view.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 "ui/message_center/views/notification_view.h"
 
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ui_resources.h"
@@ skipping 190 matching lines @@
 };
 
 ProportionalImageView::ProportionalImageView(const gfx::ImageSkia& image)
     : image_(image) {
 }
 
 ProportionalImageView::~ProportionalImageView() {
 }
 
 gfx::Size ProportionalImageView::GetPreferredSize() {
-  gfx::Size size = GetImageSizeForWidth(image_.width());
-  return gfx::Size(size.width() + GetInsets().width(),
-                   size.height() + GetInsets().height());
+  gfx::Insets insets = GetInsets();
+  gfx::Rect rect = gfx::Rect(GetImageSizeForWidth(image_.width()));
+  rect.Inset(-insets);
+  return rect.size();
 }
 
 int ProportionalImageView::GetHeightForWidth(int width) {
-  return GetImageSizeForWidth(width).height();
+  // The border will count against the width available for the image
+  // and towards the height taken by the image.
+  gfx::Insets insets = GetInsets();
+  int inset_width = width - insets.width();
+  return GetImageSizeForWidth(inset_width).height() + insets.height();
 }
 
 void ProportionalImageView::OnPaint(gfx::Canvas* canvas) {
   views::View::OnPaint(canvas);
 
-  gfx::Size draw_size(GetImageSizeForWidth(width()));
+  gfx::Size draw_size(GetImageSizeForWidth(width() - GetInsets().width()));
   if (!draw_size.IsEmpty()) {
     gfx::Rect draw_bounds = GetContentsBounds();
     draw_bounds.ClampToCenteredSize(draw_size);
 
     gfx::Size image_size(image_.size());
     if (image_size == draw_size) {
       canvas->DrawImageInt(image_, draw_bounds.x(), draw_bounds.y());
     } else {
       // Resize case
       SkPaint paint;
       paint.setFilterBitmap(true);
       canvas->DrawImageInt(image_, 0, 0,
                            image_size.width(), image_size.height(),
                            draw_bounds.x(), draw_bounds.y(),
                            draw_size.width(), draw_size.height(),
                            true, paint);
     }
   }
 }
 
 gfx::Size ProportionalImageView::GetImageSizeForWidth(int width) {
   gfx::Size size = visible() ? image_.size() : gfx::Size();
   return message_center::GetImageSizeForWidth(width, size);
 }
 
+// NotificationImage ///////////////////////////////////////////////////////
+
+// The NotificationImage is the view representing the area covered by the
+// notification's image, including background and border.  Its size can be
+// specified in advance and images will be scaled to fit including a border if
+// necessary.
+class NotificationImage : public views::View {

[Dmitry Titov, 2013/12/06 19:29:11]

Is it possible to avoid creating new class?
Can we get similar effect by setting a border, background and size on the
ProportionalImageView? Even if we need an additional background view, it can be
created when NotificaitonView's view tree is constructed. Having a separate
class does not have much value, since there is no methods to call later
(GetPreferredSize can be avoided by SetPreferredSize I think) nor specific
lifetime...

[dewittj, 2013/12/10 02:47:04]

Done.

+ public:
+  NotificationImage(const gfx::Image& image, gfx::Size size);
+  virtual ~NotificationImage();
+
+ private:
+  virtual gfx::Size GetPreferredSize() OVERRIDE;
+
+  gfx::Image image_;
+  gfx::Size size_;
+};

[Dmitry Titov, 2013/12/06 19:29:11]

DISALLOW_COPY_AND_ASSIGN...

[dewittj, 2013/12/10 02:47:04]

Done.

+
+NotificationImage::NotificationImage(const gfx::Image& image, gfx::Size size)
+    : image_(image), size_(size) {
+  SetLayoutManager(new views::FillLayout());
+  set_background(views::Background::CreateSolidBackground(
+      message_center::kImageBackgroundColor));
+  gfx::Size image_size = image_.Size();
+  float image_aspect = 0;
+  if (image_size.height() > 0)
+    image_aspect = static_cast<float>(image_size.width()) / image_size.height();
+
+  views::View* proportional_image_view =
+      new ProportionalImageView(image.AsImageSkia());
+
+  // This calculation determines that the new image would have the correct
+  // height for width.
+  if (static_cast<int>(image_aspect * size_.height()) != size_.width()) {
+    proportional_image_view->set_border(views::Border::CreateSolidBorder(
+        message_center::kNotificationImageBorderSize, SK_ColorTRANSPARENT));
+  }
+
+  AddChildView(proportional_image_view);
+}
+
+NotificationImage::~NotificationImage() {}
+
+gfx::Size NotificationImage::GetPreferredSize() { return size_; }
+
 // NotificationProgressBar /////////////////////////////////////////////////////
 
 class NotificationProgressBar : public views::ProgressBar {
  public:
   NotificationProgressBar();
   virtual ~NotificationProgressBar();
 
  private:
   // Overriden from View
   virtual gfx::Size GetPreferredSize() OVERRIDE;
@@ skipping 317 matching lines @@
 
   // Create the bottom_view_, which collects into a vertical box all content
   // below the notification icon except for the expand button.
   bottom_view_ = new views::View();
   bottom_view_->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
 
   // Create the image view if appropriate.
   image_view_ = NULL;
   if (!notification.image().IsEmpty()) {
-    image_view_ = new ProportionalImageView(notification.image().AsImageSkia());
+    gfx::Size image_size(
+        kNotificationPreferredImageSize,
+        kNotificationPreferredImageSize * kNotificationPreferredImageRatio);
+    image_view_ = new NotificationImage(notification.image(), image_size);
     image_view_->SetVisible(is_expanded_);
     bottom_view_->AddChildView(image_view_);
   }
 
   // Create action buttons if appropriate.
   std::vector<ButtonInfo> buttons = notification.buttons();
   for (size_t i = 0; i < buttons.size(); ++i) {
     views::View* separator = new views::ImageView();
     separator->set_border(MakeSeparatorBorder(1, 0, kButtonSeparatorColor));
     bottom_view_->AddChildView(separator);
@@ skipping 235 matching lines @@
   return message_view_ ?
          message_view_->GetLinesForWidthAndLimit(width, limit) : 0;
 }
 
 int NotificationView::GetMessageHeight(int width, int limit) {
   return message_view_ ?
          message_view_->GetSizeForWidthAndLines(width, limit).height() : 0;
 }
 
 }  // namespace message_center