Replace (most) occurrences of mojo::Array<T>() with nullptr.

apps/moterm/moterm_driver.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 "apps/moterm/moterm_driver.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/logging.h"
@@ skipping 182 matching lines @@
   callback.Run(mojo::files::Error::OK);
 
   client_->OnClosed();
 }
 
 void MotermDriver::Read(uint32_t num_bytes_to_read,
                         int64_t offset,
                         mojo::files::Whence whence,
                         const ReadCallback& callback) {
   if (is_closed_) {
-    callback.Run(mojo::files::Error::CLOSED, mojo::Array<uint8_t>());
+    callback.Run(mojo::files::Error::CLOSED, nullptr);
     return;
   }
 
   if (offset != 0 || whence != mojo::files::Whence::FROM_CURRENT) {
     // TODO(vtl): Is this the "right" behavior?
-    callback.Run(mojo::files::Error::INVALID_ARGUMENT, mojo::Array<uint8_t>());
+    callback.Run(mojo::files::Error::INVALID_ARGUMENT, nullptr);
     return;
   }
 
   if (!num_bytes_to_read) {
-    callback.Run(mojo::files::Error::OK, mojo::Array<uint8_t>());
+    callback.Run(mojo::files::Error::OK, nullptr);
     return;
   }
 
   pending_read_queue_.push_back(PendingRead(num_bytes_to_read, callback));
   CompletePendingReads();
 }
 
 void MotermDriver::Write(mojo::Array<uint8_t> bytes_to_write,
                          int64_t offset,
                          mojo::files::Whence whence,
@@ skipping 145 matching lines @@
   // TODO(vtl): Is this what we want? (Also is "unavailable" right? Maybe
   // unsupported/EINVAL is better.)
   callback.Run(mojo::files::Error::UNAVAILABLE,
                mojo::ScopedSharedBufferHandle());
 }
 
 void MotermDriver::Ioctl(uint32_t request,
                          mojo::Array<uint32_t> in_values,
                          const IoctlCallback& callback) {
   if (is_closed_) {
-    callback.Run(mojo::files::Error::CLOSED, mojo::Array<uint32_t>());
+    callback.Run(mojo::files::Error::CLOSED, nullptr);
     return;
   }
 
   if (request != mojo::files::kIoctlTerminal) {
-    callback.Run(mojo::files::Error::UNIMPLEMENTED, mojo::Array<uint32_t>());
+    callback.Run(mojo::files::Error::UNIMPLEMENTED, nullptr);
     return;
   }
 
   // "Is TTY?" Yes.
   if (!in_values || !in_values.size()) {
-    callback.Run(mojo::files::Error::OK, mojo::Array<uint32_t>());
+    callback.Run(mojo::files::Error::OK, nullptr);
     return;
   }
 
   switch (in_values[0]) {
     case mojo::files::kIoctlTerminalGetSettings:
       IoctlGetSettings(in_values.Pass(), callback);
       return;
     case mojo::files::kIoctlTerminalSetSettings:
       IoctlSetSettings(in_values.Pass(), callback);
       return;
     case mojo::files::kIoctlTerminalGetWindowSize:
     case mojo::files::kIoctlTerminalSetWindowSize:
       NOTIMPLEMENTED();
-      callback.Run(mojo::files::Error::UNIMPLEMENTED, mojo::Array<uint32_t>());
+      callback.Run(mojo::files::Error::UNIMPLEMENTED, nullptr);
       return;
     default:
-      callback.Run(mojo::files::Error::UNIMPLEMENTED, mojo::Array<uint32_t>());
+      callback.Run(mojo::files::Error::UNIMPLEMENTED, nullptr);
       return;
   }
 }
 
 void MotermDriver::IoctlGetSettings(mojo::Array<uint32_t> in_values,
                                     const IoctlCallback& callback) {
   if (in_values.size() != 1u) {
-    callback.Run(mojo::files::Error::INVALID_ARGUMENT, mojo::Array<uint32_t>());
+    callback.Run(mojo::files::Error::INVALID_ARGUMENT, nullptr);
     return;
   }
 
   auto out_values = mojo::Array<uint32_t>::New(kTotalFieldCount);
 
   // TODO(vtl): Add support for various things. Also, some values should be
   // hard-coded.
 
   // iflag:
   if (icrnl_)
     out_values[kIFlagIdx] |= mojo::files::kIoctlTerminalTermiosIFlagICRNL;
 
   // oflag:
   if (onlcr_)
     out_values[kOFlagIdx] |= mojo::files::kIoctlTerminalTermiosOFlagONLCR;
 
   // lflag:
   if (icanon_)
     out_values[kLFlagIdx] |= mojo::files::kIoctlTerminalTermiosLFlagICANON;
 
   // cc:
   out_values[kVEraseIdx] = verase_;
   out_values[kVEOFIdx] = veof_;
 
   callback.Run(mojo::files::Error::OK, out_values.Pass());
 }
 
 void MotermDriver::IoctlSetSettings(mojo::Array<uint32_t> in_values,
                                     const IoctlCallback& callback) {
-  callback.Run(IoctlSetSettingsHelper(in_values.Pass()),
-               mojo::Array<uint32_t>());
+  callback.Run(IoctlSetSettingsHelper(in_values.Pass()), nullptr);
 }
 
 mojo::files::Error MotermDriver::IoctlSetSettingsHelper(
     mojo::Array<uint32_t> in_values) {
   // Note: "termios" offsets (and sizes) are increased by 1, to accomodate the
   // subrequest at index 0.
 
   // The "cc" values are optional.
   if (in_values.size() < 1 + kBaseFieldCount)
     return mojo::files::Error::INVALID_ARGUMENT;
@@ skipping 24 matching lines @@
   }
   if (1 + kVEOFIdx < in_values.size()) {
     uint32_t value = in_values[1 + kVEOFIdx];
     if (value > std::numeric_limits<uint8_t>::max())
       return mojo::files::Error::INVALID_ARGUMENT;
     veof_ = static_cast<uint8_t>(value);
   }
 
   return mojo::files::Error::OK;
 }