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

Unified Diff: chrome/browser/process_singleton.h

Issue 7945014: Applied review comments from oshima for CL 7850026. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 years, 3 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: chrome/browser/process_singleton.h
diff --git a/chrome/browser/process_singleton.h b/chrome/browser/process_singleton.h
index f511748681a22498dffba95647a275ac01439d28..c0377e26bd13e840b4192d779e080fda474ede60 100644
--- a/chrome/browser/process_singleton.h
+++ b/chrome/browser/process_singleton.h
@@ -167,6 +167,8 @@ class ProcessSingleton : public base::NonThreadSafe {
// 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_singlet
oshima 2011/09/20 04:59:17 i see. thanks.
class LinuxWatcher;
scoped_refptr<LinuxWatcher> watcher_;
#endif

Powered by Google App Engine
This is Rietveld 408576698