Emit an error log line when attempting to connect to an app at an invalid URL.

mojo/shell/desktop/mojo_main.cc

 // Copyright 2013 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 <algorithm>
 #include <iostream>
 
 #include "base/at_exit.h"
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 39 matching lines @@
 GURL GetAppURLAndSetArgs(const base::CommandLine::StringType& app_url_and_args,
                          mojo::shell::Context* context) {
   // SplitString() returns empty strings for extra delimeter characters (' ').
   std::vector<std::string> argv;
   SplitString(app_url_and_args, &argv);
   argv.erase(std::remove_if(argv.begin(), argv.end(), is_empty), argv.end());
 
   if (argv.empty())
     return GURL::EmptyGURL();
   GURL app_url(argv[0]);
+  if (!app_url.is_valid() && app_url.possibly_invalid_spec().size() == 0) {
+    LOG(ERROR) << "Error: invalid URL: " << argv[0];

[Aaron Boodman, 2014/11/20 21:53:59]

same feedback as in other file.

[azani, 2014/11/20 22:10:03]

Done.

+  }
+  DCHECK(app_url.possibly_invalid_spec().size() > 0);
   if (argv.size() > 1)
     context->application_manager()->SetArgsForURL(argv, app_url);
   return app_url;
 }
 
 void RunApps(mojo::shell::Context* context) {
   const auto& command_line = *base::CommandLine::ForCurrentProcess();
   for (const auto& arg : command_line.GetArgs())
     context->Run(GetAppURLAndSetArgs(arg, context));
 }
@@ skipping 91 matching lines @@
         if (kv.first == switches::kArgsFor)
           GetAppURLAndSetArgs(kv.second, &shell_context);
       }
 
       message_loop.PostTask(FROM_HERE, base::Bind(RunApps, &shell_context));
       message_loop.Run();
     }
   }
   return 0;
 }