Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(656)

Unified Diff: dbus/file_descriptor.cc

Issue 9700072: dbus: add support for passing file descriptors (Closed) Base URL: http://git.chromium.org/git/chromium/src@master
Patch Set: add dbus::FileDescriptor per keybuk's request Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: dbus/file_descriptor.cc
diff --git a/dbus/file_descriptor.cc b/dbus/file_descriptor.cc
new file mode 100644
index 0000000000000000000000000000000000000000..661e1d8171a5fb31da305c212c7257ea7391b569
--- /dev/null
+++ b/dbus/file_descriptor.cc
@@ -0,0 +1,15 @@
+// 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.
+
+#include "base/platform_file.h"
+#include "dbus/file_descriptor.h"
+
+namespace dbus {
+
+FileDescriptor::~FileDescriptor() {
+ if (owner_)
satorux1 2012/03/28 00:32:43 If we get rid of owner_, this becomes if (value
Sam Leffler 2012/03/28 17:28:09 Please explain how to get rid of owner_. instance
+ base::ClosePlatformFile(value_);
+}
+
+} // namespace dbus
keybuk 2012/03/27 22:36:35 nit: two spaces before //
Sam Leffler 2012/03/27 23:20:40 copied from object_path; fixed

Powered by Google App Engine
This is Rietveld 408576698