Applied review comments from oshima for CL 7850026.

chrome/browser/process_singleton.h

 // Copyright (c) 2011 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 CHROME_BROWSER_PROCESS_SINGLETON_H_
 #define CHROME_BROWSER_PROCESS_SINGLETON_H_
 #pragma once
 
 #include "build/build_config.h"
 
@@ skipping 149 matching lines @@
 
   // Path in file system to the cookie file.
   FilePath cookie_path_;
 
   // Temporary directory to hold the socket.
   ScopedTempDir socket_dir_;
 
   // Helper class for linux specific messages.  LinuxWatcher is ref counted
   // because it posts messages between threads.
 #if !defined(USE_AURA)
+  // TODO(saintlou): LinuxWatcher is defined in process_singleton_linux.cc but
+  // not in process_singleton_aura.cc.

[oshima, 2011/09/19 22:31:03]

why we don't need it on linux/aura?

[Emmanuel Saint-loubert-Bié, 2011/09/19 23:26:02]

We probably need it eventually but process_singleton_aura.cc is just a stub.

[oshima, 2011/09/20 04:59:17]

i see. thanks.

   class LinuxWatcher;
   scoped_refptr<LinuxWatcher> watcher_;
 #endif
 #elif defined(OS_MACOSX)
   // Path in file system to the lock.
   FilePath lock_path_;
 
   // File descriptor associated with the lockfile, valid between
   // |Create()| and |Cleanup()|.  Two instances cannot have a lock on
   // the same file at the same time.
   int lock_fd_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ProcessSingleton);
 };
 
 #endif  // CHROME_BROWSER_PROCESS_SINGLETON_H_