Rewrite how skydb starts prompt.cc

sky/tools/debugger/prompt/prompt.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 "base/bind.h"
 #include "base/memory/weak_ptr.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/stringprintf.h"
 #include "mojo/application/application_runner_chromium.h"
 #include "mojo/public/c/system/main.h"
 #include "mojo/public/cpp/application/application_delegate.h"
 #include "mojo/public/cpp/application/application_impl.h"
 #include "net/base/net_errors.h"
 #include "net/server/http_server.h"
 #include "net/server/http_server_request_info.h"
 #include "net/socket/tcp_server_socket.h"
 #include "services/tracing/tracing.mojom.h"
 #include "sky/tools/debugger/debugger.mojom.h"
 
 namespace sky {
 namespace debugger {
 
 class Prompt : public mojo::ApplicationDelegate, public net::HttpServer::Delegate {
  public:
   Prompt()
       : is_tracing_(false),
         weak_ptr_factory_(this) {
   }
   virtual ~Prompt() {
   }
 
  private:
   // Overridden from mojo::ApplicationDelegate:
   virtual void Initialize(mojo::ApplicationImpl* app) override {
     app->ConnectToService("mojo:tracing", &tracing_);
-    if (app->args().size() > 1)
-      url_ = app->args()[1];
-    else {
-      url_ = "https://raw.githubusercontent.com/domokit/mojo/master/sky/"
-          "examples/home.sky";
+    // app_url, command_port, url_to_load
+    if (app->args().size() < 2) {
+      LOG(ERROR) << "--args-for required to specify command_port";
+      exit(2);
     }
+
+    base::StringToUint(app->args()[1], &command_port_);
+
     scoped_ptr<net::ServerSocket> server_socket(
         new net::TCPServerSocket(NULL, net::NetLog::Source()));
-    // FIXME: This port needs to be configurable, as-is we can only run
-    // one copy of mojo_shell with sky at a time!
-    uint16 port = 7777;
-    int result = server_socket->ListenWithAddressAndPort("0.0.0.0", port, 1);
+    int result = server_socket->ListenWithAddressAndPort("0.0.0.0", command_port_, 1);
     if (result != net::OK) {
       // FIXME: Should we quit here?
-      LOG(ERROR) << "Failed to bind to port " << port
+      LOG(ERROR) << "Failed to bind to port " << command_port_
                  << " skydb commands will not work, exiting.";
       exit(2);
       return;
     }
     web_server_.reset(new net::HttpServer(server_socket.Pass(), this));
   }
 
   virtual bool ConfigureIncomingConnection(
       mojo::ApplicationConnection* connection) override {
     connection->ConnectToService(&debugger_);
-    Reload();
     return true;
   }
 
   // net::HttpServer::Delegate
   void OnConnect(int connection_id) override {
   }
 
   void OnClose(int connection_id) override {
   }
 
@@ skipping 25 matching lines @@
   void OnWebSocketMessage(
       int connection_id, const std::string& data) override {
     web_server_->Send500(connection_id, "http only");
   }
 
   void Respond(int connection_id, std::string response) {
     web_server_->Send200(connection_id, response, "text/plain");
   }
 
   void Help(std::string path, int connection_id) {
-    std::string help = "Sky Debugger\n"
+    std::string help = base::StringPrintf("Sky Debugger running on port %d\n"
         "Supported URLs:\n"
         "/toggle_tracing   -- Start/stop tracing\n"
         "/reload           -- Reload the current page\n"
         "/inspect          -- Start inspector server for current page\n"
         "/quit             -- Quit\n"
-        "/load             -- Load a new URL, url in POST body.\n";
+        "/load             -- Load a new URL, url in POST body.\n",
+        command_port_);
     if (path != "/")
       help = "Unknown path: " + path + "\n\n" + help;
     Respond(connection_id, help);
   }
 
   void Load(int connection_id, std::string url) {
     url_ = url;
     Reload();
-    Respond(connection_id, "OK\n");
+    std::string response = std::string("Loaded ") + url + "\n";
+    Respond(connection_id, response);
   }
 
   void Reload() {
     debugger_->NavigateToURL(url_);
   }
 
   void Inspect(int connection_id) {
     debugger_->InjectInspector();
     Respond(connection_id,
         "Open the following URL in Chrome:\n"
@@ skipping 16 matching lines @@
     is_tracing_ = !is_tracing_;
     Respond(connection_id, response);
   }
 
   bool is_tracing_;
   DebuggerPtr debugger_;
   tracing::TraceCoordinatorPtr tracing_;
   std::string url_;
   base::WeakPtrFactory<Prompt> weak_ptr_factory_;
   scoped_ptr<net::HttpServer> web_server_;
+  unsigned command_port_;

[jamesr, 2015/01/08 23:50:38]

please use uint32_t.  unsigned is longer to type and ambiguous about how many
bits it holds

 
   DISALLOW_COPY_AND_ASSIGN(Prompt);
 };
 
 }  // namespace debugger
 }  // namespace sky
 
 MojoResult MojoMain(MojoHandle shell_handle) {
   mojo::ApplicationRunnerChromium runner(new sky::debugger::Prompt);
   runner.set_message_loop_type(base::MessageLoop::TYPE_IO);
   return runner.Run(shell_handle);
 }