Use O_NONBLOCK when opening audio pipe on linux.

remoting/host/linux/audio_pipe_reader.cc

 // 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 "remoting/host/linux/audio_pipe_reader.h"
 
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <unistd.h>
@@ skipping 89 matching lines @@
     LOG(ERROR) << "File watcher returned an error.";
     return;
   }
 
   TryOpenPipe();
 }
 
 void AudioPipeReader::TryOpenPipe() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
-  base::File new_pipe;
-  new_pipe.Initialize(
-      pipe_path_,
-      base::File::FLAG_OPEN | base::File::FLAG_READ | base::File::FLAG_ASYNC);
+  base::File new_pipe(
+      HANDLE_EINTR(open(pipe_path_.value().c_str(), O_RDONLY | O_NONBLOCK)));
 
   // If both |pipe_| and |new_pipe| are valid then compare inodes for the two
   // file descriptors. Don't need to do anything if inode hasn't changed.
   if (new_pipe.IsValid() && pipe_.IsValid()) {
     struct stat old_stat;
     struct stat new_stat;
     if (fstat(pipe_.GetPlatformFile(), &old_stat) == 0 &&
         fstat(new_pipe.GetPlatformFile(), &new_stat) == 0 &&
         old_stat.st_ino == new_stat.st_ino) {
       return;
     }
   }
 
   file_descriptor_watcher_.StopWatchingFileDescriptor();
   timer_.Stop();
 
   pipe_ = new_pipe.Pass();
 
   if (pipe_.IsValid()) {
-    // Set O_NONBLOCK flag.
-    if (HANDLE_EINTR(fcntl(pipe_.GetPlatformFile(), F_SETFL, O_NONBLOCK)) < 0) {
-      PLOG(ERROR) << "fcntl";
-      pipe_.Close();
-      return;
-    }
-
     // Set buffer size for the pipe.
     if (HANDLE_EINTR(fcntl(
             pipe_.GetPlatformFile(), F_SETPIPE_SZ, kPipeBufferSizeBytes)) < 0) {
       PLOG(ERROR) << "fcntl";
     }
 
     WaitForPipeReadable();
   }
 }
 
@@ skipping 67 matching lines @@
       pipe_.GetPlatformFile(), false, base::MessageLoopForIO::WATCH_READ,
       &file_descriptor_watcher_, this);
 }
 
 // static
 void AudioPipeReaderTraits::Destruct(const AudioPipeReader* audio_pipe_reader) {
   audio_pipe_reader->task_runner_->DeleteSoon(FROM_HERE, audio_pipe_reader);
 }
 
 }  // namespace remoting