Use int rather than size_t to pass DIP amounts to vector icon functions.

ui/views/examples/vector_example.cc

Index: ui/views/examples/vector_example.cc
diff --git a/ui/views/examples/vector_example.cc b/ui/views/examples/vector_example.cc
index f0ee3db7344c1428f73e138536b1638ed41caf81..f34b155f17817c7b7db062eb504e468e9e788a6f 100644
--- a/ui/views/examples/vector_example.cc
+++ b/ui/views/examples/vector_example.cc
@@ -4,8 +4,6 @@
 
 #include "ui/views/examples/vector_example.h"
 
-#include <stddef.h>
-
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/macros.h"
@@ -99,7 +97,7 @@ class VectorIconGallery : public View,
   void ContentsChanged(Textfield* sender,
                        const base::string16& new_contents) override {
     if (sender == size_input_) {
-      if (base::StringToSizeT(new_contents, &size_))
+      if (base::StringToInt(new_contents, &size_))

[sky, 2016/08/23 23:53:37]

If negative is considered an error condition shouldn't this DCHECK (which is
fine, give this is an example), or show the error some way?

[Peter Kasting, 2016/08/24 02:42:00]

It looks like gfx::Size() will silently clamp negatives to zero, so I think
right now you'll get the same effect as if you entered a zero.

I figured given that this was an example, this kind of error-checking/handling
didn't really matter, but this is, I think, a behavior change from before
(sorry, I should have noticed this).  I think previously, entering a negative
would return false, so we'd just clear the input field.

So the behavior-preserving change would be to add " || (size_ < 0)" to the
condition.

Should I make this change?  Do something else?

         UpdateImage();
       else
         size_input_->SetText(base::string16());
@@ -153,7 +151,7 @@ class VectorIconGallery : public View,
   Button* file_go_button_;
 
   int vector_id_;
-  size_t size_;
+  int size_;
   SkColor color_;
 
   DISALLOW_COPY_AND_ASSIGN(VectorIconGallery);