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

Side by Side Diff: chrome/service/service_process.h

Issue 6489031: Run event executor on the ui thread to remove the need to explicitly XFlush() the XTest calls. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Removed unused gtk dependency. Created 9 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_SERVICE_SERVICE_PROCESS_H_ 5 #ifndef CHROME_SERVICE_SERVICE_PROCESS_H_
6 #define CHROME_SERVICE_SERVICE_PROCESS_H_ 6 #define CHROME_SERVICE_SERVICE_PROCESS_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 16 matching lines...) Expand all
27 27
28 // The ServiceProcess does not inherit from ChildProcess because this 28 // The ServiceProcess does not inherit from ChildProcess because this
29 // process can live independently of the browser process. 29 // process can live independently of the browser process.
30 class ServiceProcess : public CloudPrintProxy::Client, 30 class ServiceProcess : public CloudPrintProxy::Client,
31 public remoting::ChromotingHostManager::Observer { 31 public remoting::ChromotingHostManager::Observer {
32 public: 32 public:
33 ServiceProcess(); 33 ServiceProcess();
34 ~ServiceProcess(); 34 ~ServiceProcess();
35 35
36 // Initialize the ServiceProcess with the message loop that it should run on. 36 // Initialize the ServiceProcess with the message loop that it should run on.
37 bool Initialize(MessageLoop* message_loop, const CommandLine& command_line); 37 bool Initialize(MessageLoopForUI* message_loop,
38 const CommandLine& command_line);
38 bool Teardown(); 39 bool Teardown();
39 // TODO(sanjeevr): Change various parts of the code such as 40 // TODO(sanjeevr): Change various parts of the code such as
40 // net::ProxyService::CreateSystemProxyConfigService to take in 41 // net::ProxyService::CreateSystemProxyConfigService to take in
41 // MessageLoopProxy* instead of MessageLoop*. When we have done that, we can 42 // MessageLoopProxy* instead of MessageLoop*. When we have done that, we can
42 // remove the io_thread() and file_thread() accessors and replace them with 43 // remove the io_thread() and file_thread() accessors and replace them with
43 // io_message_loop_proxy() and file_message_loop_proxy() respectively. 44 // io_message_loop_proxy() and file_message_loop_proxy() respectively.
44 45
45 // Returns the thread that we perform I/O coordination on (network requests, 46 // Returns the thread that we perform I/O coordination on (network requests,
46 // communication with renderers, etc. 47 // communication with renderers, etc.
47 // NOTE: You should ONLY use this to pass to IPC or other objects which must 48 // NOTE: You should ONLY use this to pass to IPC or other objects which must
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 #if defined(ENABLE_REMOTING) 132 #if defined(ENABLE_REMOTING)
132 scoped_refptr<remoting::ChromotingHostManager> remoting_host_manager_; 133 scoped_refptr<remoting::ChromotingHostManager> remoting_host_manager_;
133 #endif 134 #endif
134 135
135 DISALLOW_COPY_AND_ASSIGN(ServiceProcess); 136 DISALLOW_COPY_AND_ASSIGN(ServiceProcess);
136 }; 137 };
137 138
138 extern ServiceProcess* g_service_process; 139 extern ServiceProcess* g_service_process;
139 140
140 #endif // CHROME_SERVICE_SERVICE_PROCESS_H_ 141 #endif // CHROME_SERVICE_SERVICE_PROCESS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698