Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Unified Diff: ui/views/controls/button/image_button.cc

Issue 10437006: Converts ui/views/controls, ui/views/examples, ui/base/models to ImageSkia (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/controls/button/image_button.cc
diff --git a/ui/views/controls/button/image_button.cc b/ui/views/controls/button/image_button.cc
index 35e4960373e93169a73437022c66c22722d2f47d..729e848be2f539f1b111be53b633b835c349ef52 100644
--- a/ui/views/controls/button/image_button.cc
+++ b/ui/views/controls/button/image_button.cc
@@ -23,7 +23,7 @@ ImageButton::ImageButton(ButtonListener* listener)
v_alignment_(ALIGN_TOP),
preferred_size_(kDefaultWidth, kDefaultHeight) {
// By default, we request that the gfx::Canvas passed to our View::OnPaint()
- // implementation is flipped horizontally so that the button's bitmaps are
+ // implementation is flipped horizontally so that the button's images are
// mirrored when the UI directionality is right-to-left.
EnableCanvasFlippingForRTLUI(true);
}
@@ -40,7 +40,7 @@ void ImageButton::SetBackground(SkColor color,
const gfx::ImageSkia* image,
const gfx::ImageSkia* mask) {
if (!image || !mask) {
- background_image_.reset();
+ background_image_ = gfx::ImageSkia();
return;
}
@@ -48,9 +48,9 @@ void ImageButton::SetBackground(SkColor color,
SkBitmapOperations::CreateButtonBackground(color, *image, *mask);
}
-void ImageButton::SetOverlayImage(const SkBitmap* image) {
+void ImageButton::SetOverlayImage(const gfx::ImageSkia* image) {
if (!image) {
- overlay_image_.reset();
+ overlay_image_ = gfx::ImageSkia();
return;
}
overlay_image_ = *image;
@@ -76,7 +76,7 @@ void ImageButton::OnPaint(gfx::Canvas* canvas) {
// Call the base class first to paint any background/borders.
View::OnPaint(canvas);
- SkBitmap img = GetImageToPaint();
+ gfx::ImageSkia img = GetImageToPaint();
if (!img.isNull()) {
int x = 0, y = 0;
@@ -105,8 +105,8 @@ void ImageButton::OnPaint(gfx::Canvas* canvas) {
////////////////////////////////////////////////////////////////////////////////
// ImageButton, protected:
-SkBitmap ImageButton::GetImageToPaint() {
- SkBitmap img;
+gfx::ImageSkia ImageButton::GetImageToPaint() {
+ gfx::ImageSkia img;
if (!images_[BS_HOT].isNull() && hover_animation_->is_animating()) {
img = SkBitmapOperations::CreateBlendedBitmap(images_[BS_NORMAL],
@@ -134,7 +134,7 @@ void ToggleImageButton::SetToggled(bool toggled) {
return;
for (int i = 0; i < BS_COUNT; ++i) {
- SkBitmap temp = images_[i];
+ gfx::ImageSkia temp = images_[i];
images_[i] = alternate_images_[i];
alternate_images_[i] = temp;
}

Powered by Google App Engine
This is Rietveld 408576698