Adjust uber tray spacing around avatar and draw avatar using rounded rectangle.

ash/system/user/tray_user.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 "ash/system/user/tray_user.h"
 
 #include "ash/shell.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/system/tray/tray_constants.h"
 #include "base/utf_string_conversions.h"
 #include "grit/ash_strings.h"
+#include "third_party/skia/include/core/SkCanvas.h"
+#include "third_party/skia/include/core/SkPaint.h"
+#include "third_party/skia/include/core/SkPath.h"
+#include "third_party/skia/include/core/SkShader.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/size.h"
+#include "ui/gfx/skia_util.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/button/text_button.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/view.h"
 
 namespace {
 
 const int kPaddingAroundButtons = 5;
 
 const int kUserInfoHorizontalPadding = 14;
 const int kUserInfoVerticalPadding = 10;
 const int kUserInfoPaddingBetweenItems = 3;
 
-const int kUserIconSize = 32;
+const int kUserIconSize = 27;
+const int kUserIconCornerRadius = 2;
 
 const SkColor kButtonStrokeColor = SkColorSetRGB(0xdd, 0xdd, 0xdd);
 
 // A custom textbutton with some extra vertical padding, and custom border,
 // alignment and hover-effects.
 class TrayButton : public views::TextButton {
  public:
   TrayButton(views::ButtonListener* listener, const string16& text)
       : views::TextButton(listener, text),
         hover_(false),
@@ skipping 211 matching lines @@
   views::Label* email_;
   views::View* update_;
 
   TrayButton* shutdown_;
   TrayButton* signout_;
   TrayButton* lock_;
 
   DISALLOW_COPY_AND_ASSIGN(UserView);
 };
 
+// A custom image view with rounded edges.
+class RoundedImageView : public views::View {
+ public:
+  // Constructs a new rounded image view with rounded corners of radius
+  // |corner_radius|.
+  RoundedImageView(int corner_radius) : corner_radius_(corner_radius) {

[Ben Goodger (Google), 2012/03/21 22:10:41]

explicit

[flackr, 2012/03/21 23:19:05]

Done.

+  }
+
+  virtual ~RoundedImageView() {
+  }
+
+  // Set the bitmap that should be displayed from a pointer. The pointer
+  // contents is copied in the receiver's bitmap.
+  void SetImage(const SkBitmap& bm) {
+    image_ = bm;
+    SetImageSize(gfx::Size(bm.width(), bm.height()));
+  }
+
+  // Set the desired image size.
+  void SetImageSize(const gfx::Size& image_size) {
+    image_size_ = image_size;
+    PreferredSizeChanged();
+  }
+
+  // Overridden from views::View.
+  virtual gfx::Size GetPreferredSize() OVERRIDE {
+    return image_size_;
+  }
+
+  virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE {
+    View::OnPaint(canvas);
+    gfx::Rect image_bounds(image_size_);
+    const SkScalar kRadius = SkIntToScalar(corner_radius_);
+    SkPath path;
+    path.addRoundRect(gfx::RectToSkRect(image_bounds), kRadius, kRadius);
+
+    SkPaint paint;
+    SkShader* shader = SkShader::CreateBitmapShader(image_,
+                                                    SkShader::kRepeat_TileMode,
+                                                    SkShader::kRepeat_TileMode);
+    SkMatrix shader_scale;
+    shader_scale.setScale(SkFloatToScalar(
+        static_cast<float>(image_bounds.width()) / image_.width()),
+        SkFloatToScalar(static_cast<float>(
+            image_bounds.height()) / image_.height()));
+    shader->setLocalMatrix(shader_scale);
+
+    paint.setShader(shader);
+    paint.setXfermodeMode(SkXfermode::kSrcOver_Mode);
+    shader->unref();
+    canvas->sk_canvas()->drawPath(path, paint);
+  }
+
+ private:
+  SkBitmap image_;
+  gfx::Size image_size_;
+  int corner_radius_;
+};

[Ben Goodger (Google), 2012/03/21 22:10:41]

DISALLOW_COPY

[flackr, 2012/03/21 23:19:05]

Done.

+
 }  // namespace tray
 
 TrayUser::TrayUser() {
 }
 
 TrayUser::~TrayUser() {
 }
 
 views::View* TrayUser::CreateTrayView(user::LoginStatus status) {
-  avatar_.reset(new views::ImageView);
+  avatar_.reset(new tray::RoundedImageView(kUserIconCornerRadius));
   if (status == user::LOGGED_IN_USER || status == user::LOGGED_IN_OWNER) {
     avatar_->SetImage(
         ash::Shell::GetInstance()->tray_delegate()->GetUserImage());
     avatar_->SetImageSize(gfx::Size(kUserIconSize, kUserIconSize));
   } else {
     avatar_->SetVisible(false);
   }
   return avatar_.get();
 }
 
@@ skipping 25 matching lines @@
     user_->RefreshForUpdate();
 }
 
 void TrayUser::OnUserUpdate() {
   avatar_->SetImage(
       ash::Shell::GetInstance()->tray_delegate()->GetUserImage());
 }
 
 }  // namespace internal
 }  // namespace ash