net: Remove deprecated IOBufferWithSize constructors.

Bug: 488553
Change-Id: Id732ac4eb5e5afc26095a925e73797449369c492
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2371471
Auto-Submit: Victor Costan <pwnall@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#801175}

net/base/io_buffer.cc

@@ -43,22 +43,10 @@ IOBuffer::~IOBuffer() {
   data_ = nullptr;
 }
 
-IOBufferWithSize::IOBufferWithSize(int size)
-    : IOBuffer(size),
-      size_(size) {
-  AssertValidBufferSize(size);
-}
-
 IOBufferWithSize::IOBufferWithSize(size_t size) : IOBuffer(size), size_(size) {
   // Note: Size check is done in superclass' constructor.
 }
 
-IOBufferWithSize::IOBufferWithSize(char* data, int size)
-    : IOBuffer(data),
-      size_(size) {
-  AssertValidBufferSize(size);
-}
-
 IOBufferWithSize::IOBufferWithSize(char* data, size_t size)
     : IOBuffer(data), size_(size) {
   AssertValidBufferSize(size);

net/base/io_buffer.h

@@ -104,16 +104,11 @@ class NET_EXPORT IOBuffer : public base::RefCountedThreadSafe<IOBuffer> {
 // argument to IO functions. Please keep using IOBuffer* for API declarations.
 class NET_EXPORT IOBufferWithSize : public IOBuffer {
  public:
-  // TODO(eroman): Deprecated. Use the size_t flavor instead. crbug.com/488553
-  explicit IOBufferWithSize(int size);
   explicit IOBufferWithSize(size_t size);
 
   int size() const { return size_; }
 
  protected:
-  // TODO(eroman): Deprecated. Use the size_t flavor instead. crbug.com/488553
-  IOBufferWithSize(char* data, int size);
-
   // Purpose of this constructor is to give a subclass access to the base class
   // constructor IOBuffer(char*) thus allowing subclass to use underlying
   // memory it does not own.