Remove callers of mojo::Array<size_t> constructor in favor of ::New

services/files/file_impl.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 "services/files/file_impl.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <sys/types.h>
@@ skipping 80 matching lines @@
     // multithreaded.) Maybe we should do an |ftell()| and always use |pread()|.
     // TODO(vtl): Possibly, at least sometimes we should not change the file
     // position. See TODO in file.mojom.
     if (lseek(file_fd_.get(), static_cast<off_t>(offset),
               WhenceToStandardWhence(whence)) < 0) {
       callback.Run(ErrnoToError(errno), Array<uint8_t>());
       return;
     }
   }
 
-  Array<uint8_t> bytes_read(num_bytes_to_read);
+  auto bytes_read = Array<uint8_t>::New(num_bytes_to_read);
   ssize_t num_bytes_read = HANDLE_EINTR(
       read(file_fd_.get(), &bytes_read.front(), num_bytes_to_read));
   if (num_bytes_read < 0) {
     callback.Run(ErrnoToError(errno), Array<uint8_t>());
     return;
   }
 
   DCHECK_LE(static_cast<size_t>(num_bytes_read), num_bytes_to_read);
   bytes_read.resize(static_cast<size_t>(num_bytes_read));
   callback.Run(Error::OK, bytes_read.Pass());
@@ skipping 236 matching lines @@
     return;
   }
 
   // TODO(vtl): The "correct" error code should be one that can be translated to
   // ENOTTY!
   callback.Run(Error::UNAVAILABLE, Array<uint32_t>());
 }
 
 }  // namespace files
 }  // namespace mojo