Fix some potential Mach port leaks from mach_host_self using a new ScopedMachPort class.

base/mac/scoped_mach_port.h

Index: base/mac/scoped_mach_port.h
diff --git a/base/mac/scoped_mach_port.h b/base/mac/scoped_mach_port.h
new file mode 100644
index 0000000000000000000000000000000000000000..bb761a73c9dd636742219b892c0fddbbda329d55
--- /dev/null
+++ b/base/mac/scoped_mach_port.h
@@ -0,0 +1,40 @@
+// Copyright (c) 2012 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 BASE_MAC_SCOPED_MACH_PORT_H_
+#define BASE_MAC_SCOPED_MACH_PORT_H_
+
+#include <mach/mach.h>
+
+#include "base/basictypes.h"
+#include "base/base_export.h"
+
+namespace base {
+namespace mac {
+
+// A class for managing the life of a Mach port, releasing via
+// mach_port_deallocate either its send and/or receive rights.
+class BASE_EXPORT ScopedMachPort {
+ public:
+  // Creates a scoper with MACH_PORT_NULL.

[Mark Mentovai, 2012/10/16 16:34:28]

This is unused and can be gotten rid of.

[Robert Sesek, 2012/10/16 16:40:23]

Done.

+  ScopedMachPort();
+
+  // Creates a scoper by taking ownership of the port.
+  explicit ScopedMachPort(mach_port_t port);
+
+  ~ScopedMachPort();
+
+  operator mach_port_t() const;

[Mark Mentovai, 2012/10/16 16:34:28]

These two getters should remain inlined in the header. Accessing the field is
cheaper than calling (execution-wise) and just as small (code size-wise).

[Robert Sesek, 2012/10/16 16:40:23]

Done.

+  mach_port_t get() const;
+
+ private:
+  mach_port_t port_;
+
+  DISALLOW_COPY_AND_ASSIGN(ScopedMachPort);
+};
+
+}  // namespace mac
+}  // namespace base
+
+#endif  // BASE_MAC_SCOPED_MACH_PORT_H_