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

ui/views/examples/vector_example.cc

 // Copyright 2015 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.
 
 #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"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/vector_icons_public.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/button/blue_button.h"
@@ skipping 73 matching lines @@
       UpdateImage();
       return true;
     }
     return false;
   }
 
   // TextfieldController implementation.
   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());
 
       return;
     }
 
     DCHECK_EQ(color_input_, sender);
     if (new_contents.size() != 8u)
       return;
@@ skipping 33 matching lines @@
 
  private:
   ImageView* image_view_;
   View* image_view_container_;
   Textfield* size_input_;
   Textfield* color_input_;
   Textfield* file_chooser_;
   Button* file_go_button_;
 
   int vector_id_;
-  size_t size_;
+  int size_;
   SkColor color_;
 
   DISALLOW_COPY_AND_ASSIGN(VectorIconGallery);
 };
 
 }  // namespace
 
 VectorExample::VectorExample() : ExampleBase("Vector Icon") {}
 
 VectorExample::~VectorExample() {}
 
 void VectorExample::CreateExampleView(View* container) {
   container->SetLayoutManager(new FillLayout());
   container->AddChildView(new VectorIconGallery());
 }
 
 }  // namespace examples
 }  // namespace views