Initialize IPC:ChannelHandle from existing HANDLE

base/win/scoped_handle.h

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.

[cpu_(ooo_6.6-7.5), 2012/01/19 20:56:05]

2012

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_WIN_SCOPED_HANDLE_H_
 #define BASE_WIN_SCOPED_HANDLE_H_
 #pragma once
 
 #include <windows.h>
 
 #include "base/basictypes.h"
@@ skipping 38 matching lines @@
   }
 
   void Set(HANDLE new_handle) {
     Close();
 
     // Windows is inconsistent about invalid handles, so we always use NULL
     if (new_handle != INVALID_HANDLE_VALUE)
       handle_ = new_handle;
   }
 
-  HANDLE Get() {
+  HANDLE Get() const {
     return handle_;
   }
 
-  operator HANDLE() { return handle_; }
+  operator HANDLE() const { return handle_; }

[cpu_(ooo_6.6-7.5), 2012/01/19 20:56:05]

can you make it so it looks like Get()? line-wise I mean.

 
   HANDLE Take() {
     // transfers ownership away from this object
     HANDLE h = handle_;
     handle_ = NULL;
     return h;
   }
 
   void Close() {
     if (handle_) {
       if (!::CloseHandle(handle_)) {
         NOTREACHED();
       }
       handle_ = NULL;
     }
   }
 

[cpu_(ooo_6.6-7.5), 2012/01/19 20:56:05]

can you add a method HANDLE Duplicate() const that does like:

*out_handle = NULL or INVALID_HANDLE_VALUE;
DuplicateHandle(GetCurrentProcess(),
                handle,
                GetCurrentProcess(),
                out_handle_,
                0,
                FALSE,
                DUPLICATE_SAME_ACCESS);

  private:
   HANDLE handle_;
   DISALLOW_COPY_AND_ASSIGN(ScopedHandle);
 };
 
 }  // namespace win
 }  // namespace base
 
 #endif  // BASE_SCOPED_HANDLE_WIN_H_