Adds ScopedHandleBase::From

mojo/public/cpp/system/core.h

 // Copyright 2013 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.
 
 #ifndef MOJO_PUBLIC_CPP_SYSTEM_CORE_H_
 #define MOJO_PUBLIC_CPP_SYSTEM_CORE_H_
 
 #include <assert.h>
 #include <stddef.h>
 
@@ skipping 31 matching lines @@
 
   // Move-only constructor and operator=.
   ScopedHandleBase(RValue other) : handle_(other.object->release()) {}
   ScopedHandleBase& operator=(RValue other) {
     handle_ = other.object->release();
     return *this;
   }
 
   const HandleType& get() const { return handle_; }
 
+  template <typename Passed>

[viettrungluu, 2014/04/03 20:42:48]

Nit: Could you call this PassedHandleType?

+  static ScopedHandleBase<HandleType> From(ScopedHandleBase<Passed> other) {
+    return ScopedHandleBase<HandleType>(

[viettrungluu, 2014/04/03 20:42:48]

Please add a comment:
// The |static_cast| here is needed to ensure type-safety (namely,
// that |HandleType| is a |PassedHandleType|).

+        static_cast<HandleType>(other.release().value()));

[viettrungluu, 2014/04/03 20:42:48]

It should be static_cast<HandleType>(other.release()).value()

+  }
+
   void swap(ScopedHandleBase& other) {
     handle_.swap(other.handle_);
   }
 
   HandleType release() MOJO_WARN_UNUSED_RESULT {
     HandleType rv;
     rv.swap(handle_);
     return rv;
   }
 
@@ skipping 396 matching lines @@
       CreateSharedBuffer(&options, num_bytes, &handle);
   assert(result == MOJO_RESULT_OK);
 }
 
 inline SharedBuffer::~SharedBuffer() {
 }
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_SYSTEM_CORE_H_