Make ShadowBorder use ShadowValue.

ui/views/shadow_border.h

Index: ui/views/shadow_border.h
diff --git a/ui/views/shadow_border.h b/ui/views/shadow_border.h
index fd0565f11ff5e4f0940901106e2a8602a7e67c92..be8e53bc6c4aef32bf1af78ae6f3d23a9550dc1d 100644
--- a/ui/views/shadow_border.h
+++ b/ui/views/shadow_border.h
@@ -6,6 +6,7 @@
 #define UI_VIEWS_SHADOW_BORDER_H_
 
 #include "third_party/skia/include/core/SkColor.h"
+#include "ui/gfx/shadow_value.h"
 #include "ui/views/border.h"
 #include "ui/views/views_export.h"
 
@@ -18,6 +19,7 @@ class VIEWS_EXPORT ShadowBorder : public views::Border {
                SkColor color,
                int vertical_offset,
                int horizontal_offset);
+  ShadowBorder(const gfx::ShadowValue& shadow);

[sky, 2015/02/05 17:33:32]

I don't want another constructor that effectively duplicates the existing one.
I'm ok with this if you plan on going through and fixing all callers to the new
one. Is that your intention? Also, this should be explicit.

[calamity, 2015/02/06 05:29:53]

Yeah, I'll just add it to this CL. Only a couple of callers.

   ~ShadowBorder() override;
 
  protected:
@@ -27,18 +29,8 @@ class VIEWS_EXPORT ShadowBorder : public views::Border {
   gfx::Size GetMinimumSize() const override;
 
  private:
-  // Blur amount of the shadow in pixels. For details on how blur is defined see
-  // comments for blur_ in class ShadowValue.
-  const int blur_;
-
-  // Shadow color.
-  const SkColor color_;
-
-  // Number of pixels to shift shadow to bottom.
-  const int vertical_offset_;
-
-  // Number of pixels to shift shadow to right.
-  const int horizontal_offset_;
+  // The shadow value to use for this border.
+  const gfx::ShadowValue shadow_value_;
 
   DISALLOW_COPY_AND_ASSIGN(ShadowBorder);
 };