Clean up SkBitmapOperations.

ui/app_list/app_list_item_view.cc

Index: ui/app_list/app_list_item_view.cc
diff --git a/ui/app_list/app_list_item_view.cc b/ui/app_list/app_list_item_view.cc
index f3e6803759947bf4fe7fc70cffd05c5fba257a7f..5e7e4fc96873f51222f31bc6e37f348ff3753201 100644
--- a/ui/app_list/app_list_item_view.cc
+++ b/ui/app_list/app_list_item_view.cc
@@ -5,12 +5,14 @@
 #include "ui/app_list/app_list_item_view.h"
 
 #include <algorithm>
+#include <vector>
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/synchronization/cancellation_flag.h"
 #include "base/threading/worker_pool.h"
 #include "base/utf_string_conversions.h"
+#include "skia/ext/image_operations.h"
 #include "ui/app_list/app_list_item_model.h"
 #include "ui/app_list/apps_grid_view.h"
 #include "ui/app_list/drop_shadow_label.h"
@@ -126,39 +128,29 @@ class AppListItemView::IconOperation
   static const int kShadowPadding = 15;
 
   void ResizeAndGenerateShadow() {
-    // If you change shadow radius and shadow offset, please also update
-    // kShadowPaddingAbove.
-    const SkColor kShadowColor[] = {
-      SkColorSetARGB(0xCC, 0, 0, 0),
-      SkColorSetARGB(0x33, 0, 0, 0),
-      SkColorSetARGB(0x4C, 0, 0, 0),
-    };
-    const gfx::Point kShadowOffset[] = {
-      gfx::Point(0, 0),
-      gfx::Point(0, 4),
-      gfx::Point(0, 5),
-    };
-    const SkScalar kShadowRadius[] = {
-      SkIntToScalar(2),
-      SkIntToScalar(4),
-      SkIntToScalar(10),
-    };
-
     if (cancel_flag_.IsSet())
       return;
 
-    if (size_ != gfx::Size(bitmap_.width(), bitmap_.height()))
-      bitmap_ = SkBitmapOperations::CreateResizedBitmap(bitmap_, size_);
+    if (size_ != gfx::Size(bitmap_.width(), bitmap_.height())) {
+      bitmap_ = skia::ImageOperations::Resize(bitmap_,
+          skia::ImageOperations::RESIZE_BEST, size_.width(), size_.height());
+    }
 
     if (cancel_flag_.IsSet())
       return;
 
-    bitmap_ = SkBitmapOperations::CreateDropShadow(
-        bitmap_,
-        arraysize(kShadowColor),
-        kShadowColor,
-        kShadowOffset,
-        kShadowRadius);
+    // If you change shadow radius and shadow offset, please also update
+    // kShadowPaddingAbove.
+    const gfx::ShadowValue kShadows[] = {
+      gfx::ShadowValue(gfx::Point(0, 0), 2, SkColorSetARGB(0xCC, 0, 0, 0)),
+      gfx::ShadowValue(gfx::Point(0, 4), 4, SkColorSetARGB(0x33, 0, 0, 0)),
+      gfx::ShadowValue(gfx::Point(0, 5), 10, SkColorSetARGB(0x4C, 0, 0, 0)),
+    };
+    const std::vector<gfx::ShadowValue> shadows(
+        &kShadows[0],
+        &kShadows[arraysize(kShadows)]);

[Alexei Svitkine (slow), 2012/05/25 18:16:06]

I think initializing with (kShadows, kShadows + arraysize(kShadows)) is a bit
cleaner. Also, maybe just inline it into the call below.

[xiyuan, 2012/05/25 19:34:01]

Done.

+
+    bitmap_ = SkBitmapOperations::CreateDropShadow(bitmap_, shadows);
   }
 
   void Cancel() {