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

Unified Diff: chromeos/dbus/debug_daemon_client.cc

Issue 10701050: net: Implement canceling of all async operations in FileStream. (Closed) Base URL: https://src.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 1 month 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: chromeos/dbus/debug_daemon_client.cc
diff --git a/chromeos/dbus/debug_daemon_client.cc b/chromeos/dbus/debug_daemon_client.cc
index 05d3b394ddce75695455ce808f9ccd1209c04f3f..0b509ffcb0efca9fb98b20dd4809e22e3e919c97 100644
--- a/chromeos/dbus/debug_daemon_client.cc
+++ b/chromeos/dbus/debug_daemon_client.cc
@@ -51,9 +51,7 @@ class PipeReader {
}
virtual ~PipeReader() {
- if (pipe_fd_[0] != -1)
- if (HANDLE_EINTR(close(pipe_fd_[0])) < 0)
- PLOG(ERROR) << "close[0]";
+ // Don't close pipe_fd_[0] as it's closed by data_stream_.
if (pipe_fd_[1] != -1)
if (HANDLE_EINTR(close(pipe_fd_[1])) < 0)
PLOG(ERROR) << "close[1]";

Powered by Google App Engine
This is Rietveld 408576698