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

Unified Diff: remoting/host/me2me_preference_pane.mm

Issue 10855249: [Chromoting] The daemon process now starts the networking process and passes the host configuration… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added missing signal.h Created 8 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
« no previous file with comments | « remoting/host/json_host_config.cc ('k') | remoting/host/remoting_me2me_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/me2me_preference_pane.mm
diff --git a/remoting/host/me2me_preference_pane.mm b/remoting/host/me2me_preference_pane.mm
index bd8239616e226c8bef4cb396bc3b1eced440ff03..dcb35931f002327181cef95d5907bf615ae639f5 100644
--- a/remoting/host/me2me_preference_pane.mm
+++ b/remoting/host/me2me_preference_pane.mm
@@ -524,17 +524,8 @@ std::string JsonHostConfig::GetSerializedData() const {
have_new_config_ = NO;
- // If the service is running, send a signal to cause it to reload its
- // configuration, otherwise start the service.
- if (is_service_running_) {
- pid_t job_pid = base::mac::PIDForJob(remoting::kServiceName);
- if (job_pid > 0) {
- kill(job_pid, SIGHUP);
- } else {
- NSLog(@"Failed to obtain PID of service %s", remoting::kServiceName);
- [self showError];
- }
- } else {
+ // Ensure the service is started.
+ if (!is_service_running_) {
[self sendJobControlMessage:LAUNCH_KEY_STARTJOB];
}
« no previous file with comments | « remoting/host/json_host_config.cc ('k') | remoting/host/remoting_me2me_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698