[mojo-edk] Add explicit message object APIs

mojo/edk/js/core.cc

 // Copyright 2014 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 "mojo/edk/js/core.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
@@ skipping 127 matching lines @@
     MojoWriteMessageFlags flags) {
   std::vector<MojoHandle> raw_handles(handles.size());
   for (size_t i = 0; i < handles.size(); ++i)
     raw_handles[i] = handles[i]->get().value();
   MojoResult rv = MojoWriteMessage(handle.value(),
                           buffer.bytes(),
                           static_cast<uint32_t>(buffer.num_bytes()),
                           raw_handles.empty() ? NULL : &raw_handles[0],
                           static_cast<uint32_t>(raw_handles.size()),
                           flags);
-  // MojoWriteMessage takes ownership of the handles upon success, so
-  // release them here.
-  if (rv == MOJO_RESULT_OK) {
-    for (size_t i = 0; i < handles.size(); ++i)
-      ignore_result(handles[i]->release());
-  }
+  // MojoWriteMessage takes ownership of the handles, so release them here.
+  for (size_t i = 0; i < handles.size(); ++i)
+    ignore_result(handles[i]->release());
+
   return rv;
 }
 
 gin::Dictionary ReadMessage(const gin::Arguments& args,
                             mojo::Handle handle,
                             MojoReadMessageFlags flags) {
   uint32_t num_bytes = 0;
   uint32_t num_handles = 0;
   MojoResult result = MojoReadMessage(
       handle.value(), NULL, &num_bytes, NULL, &num_handles, flags);
@@ skipping 211 matching lines @@
 
     data->SetObjectTemplate(&g_wrapper_info, templ);
   }
 
   return templ->NewInstance();
 }
 
 }  // namespace js
 }  // namespace edk
 }  // namespace mojo