Set RemoteDebuggingServer port based on "remote-debugging-port" flag.

chromecast/browser/devtools/remote_debugging_server.cc

 // Copyright 2014 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 "chromecast/browser/devtools/remote_debugging_server.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
+#include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "chromecast/browser/cast_browser_process.h"
 #include "chromecast/browser/devtools/cast_dev_tools_delegate.h"
 #include "chromecast/common/cast_content_client.h"
 #include "chromecast/common/pref_names.h"
 #include "components/devtools_http_handler/devtools_http_handler.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/user_agent.h"
@@ skipping 84 matching lines @@
       new TCPServerSocketFactory("0.0.0.0", port));
 #endif
 }
 
 std::string GetFrontendUrl() {
   return base::StringPrintf(kFrontEndURL, content::GetWebKitRevision().c_str());
 }
 
 }  // namespace
 
-RemoteDebuggingServer::RemoteDebuggingServer() : port_(0) {
+RemoteDebuggingServer::RemoteDebuggingServer()
+    : port_(kDefaultRemoteDebuggingPort) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  pref_port_.Init(prefs::kRemoteDebuggingPort,
-                  CastBrowserProcess::GetInstance()->pref_service(),
-                  base::Bind(&RemoteDebuggingServer::OnPortChanged,
-                             base::Unretained(this)));
+  pref_enabled_.Init(prefs::kEnableRemoteDebugging,
+                     CastBrowserProcess::GetInstance()->pref_service(),
+                     base::Bind(&RemoteDebuggingServer::OnEnabledChanged,
+                                base::Unretained(this)));
+
+  std::string port_str =
+      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kRemoteDebuggingPort);
+  if (!port_str.empty()) {
+    int port = kDefaultRemoteDebuggingPort;
+    if (base::StringToInt(port_str, &port)) {
+      port_ = static_cast<uint16>(port);
+    } else {
+      port_ = kDefaultRemoteDebuggingPort;
+    }
+  }
 
   // Starts new dev tools, clearing port number saved in config.
   // Remote debugging in production must be triggered only by config server.
-  pref_port_.SetValue(ShouldStartImmediately() ?
-                      kDefaultRemoteDebuggingPort : 0);
-  OnPortChanged();
+  pref_enabled_.SetValue(ShouldStartImmediately() && port_ != 0);
+  OnEnabledChanged();
 }
 
 RemoteDebuggingServer::~RemoteDebuggingServer() {
-  pref_port_.SetValue(0);
-  OnPortChanged();
+  pref_enabled_.SetValue(false);
+  OnEnabledChanged();
 }
 
-void RemoteDebuggingServer::OnPortChanged() {
-  uint16 new_port = static_cast<uint16>(std::max(*pref_port_, 0));
-  VLOG(1) << "OnPortChanged called: old_port=" << port_
-          << ", new_port=" << new_port;
-
-  if (new_port == port_) {
-    VLOG(1) << "Port has not been changed. Ignore silently.";
-    return;
-  }
-
-  if (devtools_http_handler_) {
-    LOG(INFO) << "Stop old devtools: port=" << port_;
-    devtools_http_handler_.reset();
-  }
-
-  port_ = new_port;
-  if (port_ > 0) {
+void RemoteDebuggingServer::OnEnabledChanged() {
+  bool enabled = *pref_enabled_ && port_ != 0;
+  if (enabled && !devtools_http_handler_) {
     devtools_http_handler_.reset(new DevToolsHttpHandler(
         CreateSocketFactory(port_),
         GetFrontendUrl(),
         new CastDevToolsDelegate(),
         base::FilePath(),
         base::FilePath(),
         std::string(),
         GetUserAgent()));
     LOG(INFO) << "Devtools started: port=" << port_;
+  } else if (devtools_http_handler_) {

[byungchul, 2015/06/16 16:57:24]

it includes the condition "enabled && evtools_http_handler_" though it might be
fine because OnEnabledChanged() will be called only when enabled changed. But,
still good to add !enabled.

[derekjchow1, 2015/06/16 17:02:48]

Done.

+    LOG(INFO) << "Stop devtools: port=" << port_;
+    devtools_http_handler_.reset();
   }
 }
 
 }  // namespace shell
 }  // namespace chromecast