Change Mojo URLs to structured names

mojo/public/tools/manifest/manifest_collator.py

 #!/usr/bin/env python
 # Copyright 2016 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.
 
 """ A collator for Mojo Application Manifests """
 
 import argparse
 import json
 import shutil
@@ skipping 13 matching lines @@
       description="Collate Mojo application manifests.")
   parser.add_argument("--parent")
   parser.add_argument("--output")
   parser.add_argument("--application-name")
   args, children = parser.parse_known_args()
 
   parent = ParseJSONFile(args.parent)
   if parent == None:
     return 1
 
-  parsed = urlparse.urlparse(parent['url'])
-  if args.application_name != parsed.hostname:
+  app_path = parent['name'].split(':')[1]

[sky, 2016/02/26 18:37:27]

Error checking if split(':') returns a list of one element? Also, what about
error checking that app_path doesn't start with '//'?

+  if args.application_name != app_path:
     raise ValueError("Application name '%s' specified in build file does not " \
                      "match application name '%s' specified in manifest." %
-                     (args.application_name, parsed.hostname))
+                     (args.application_name, app_path))
 
   applications = []
   for child in children:
     application = ParseJSONFile(child)
     if application == None:
       return 1
     applications.append(application)
 
   if len(applications) > 0:
     parent['applications'] = applications
 
   with open(args.output, 'w') as output_file:
     json.dump(parent, output_file)
 
   return 0
 
 if __name__ == "__main__":
   sys.exit(main())