Clean up SkBitmapOperations.

ui/gfx/skbitmap_operations.h

 // 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.
 
 #ifndef UI_GFX_SKBITMAP_OPERATIONS_H_
 #define UI_GFX_SKBITMAP_OPERATIONS_H_
 #pragma once
 
+#include <vector>
+
 #include "base/gtest_prod_util.h"
 #include "ui/base/ui_export.h"
 #include "ui/gfx/color_utils.h"
 
 namespace gfx {
 class Point;
+class ShadowValue;
 class Size;
 }
 
 class SkBitmap;
 
 class UI_EXPORT SkBitmapOperations {
  public:
   // Create a bitmap that is an inverted image of the passed in image.
   // Each color becomes its inverse in the color wheel. So (255, 15, 0) becomes
   // (0, 240, 255). The alpha value is not inverted.
@@ skipping 66 matching lines @@
 
   // Unpremultiplies all pixels in |bitmap|. You almost never want to call
   // this, as |SkBitmap|s are always premultiplied by conversion. Call this
   // only if you will pass the bitmap's data into a system function that
   // doesn't expect premultiplied colors.
   static SkBitmap UnPreMultiply(const SkBitmap& bitmap);
 
   // Transpose the pixels in |bitmap| by swapping x and y.
   static SkBitmap CreateTransposedBtmap(const SkBitmap& bitmap);
 
-  // Create a copy of |bitmap| with specified |size|. The image must use the
-  // kARGB_8888_Config config.
-  static SkBitmap CreateResizedBitmap(const SkBitmap& bitmap,
-                                      const gfx::Size& size);
-
   // Create a bitmap by combining alpha channel of |bitmap| and color |c|.
   // The image must use the kARGB_8888_Config config.
   static SkBitmap CreateColorMask(const SkBitmap& bitmap, SkColor c);
 
   // Create a bitmap with drop shadow added to |bitmap|. |shadow_count| is the
   // number of shadows to add. |shadow_offset| and |shadow_radius| are arrays
   // with |shadow_count| elements to provide definition for each shadow. The
   // created bitmap would be padded to have enough space for shadows and have
   // original bitmap in the center. The image must use the kARGB_8888_Config
   // config.

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

Please update the comment (it references things like |shadow_count| that no
longer exist).

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

Done. Totally blind. Thanks for pointing out. :p

-  static SkBitmap CreateDropShadow(const SkBitmap& bitmap,
-                                   int shadow_count,
-                                   const SkColor* shadow_color,
-                                   const gfx::Point* shadow_offset,
-                                   const SkScalar* shadow_radius);
+  static SkBitmap CreateDropShadow(
+      const SkBitmap& bitmap,
+      const std::vector<gfx::ShadowValue>& shadows);
+
  private:
   SkBitmapOperations();  // Class for scoping only.
 
   FRIEND_TEST_ALL_PREFIXES(SkBitmapOperationsTest, DownsampleByTwo);
   FRIEND_TEST_ALL_PREFIXES(SkBitmapOperationsTest, DownsampleByTwoSmall);
 };
 
 #endif  // UI_GFX_SKBITMAP_OPERATIONS_H_