Fixing json compiler preview.

tools/json_schema_compiler/schema_loader.py

 # Copyright (c) 2012 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.
 
 import os
 import re
 import sys
 
 import idl_schema
 import json_schema
@@ skipping 32 matching lines @@
   def __init__(self,
                display_path,
                real_path,
                include_rules,
                cpp_namespace_pattern):
     self._display_path = display_path
     self._real_path = real_path
     self._include_rules = [(real_path, cpp_namespace_pattern)]
     self._include_rules.extend(include_rules)
 
-  def ResolveType(self, full_name, default_namespace):
-    name_parts = full_name.rsplit('.', 1)
-    if len(name_parts) == 1:
-      if full_name not in default_namespace.types:
-        return None
-      return default_namespace
-    full_namespace, type_name = full_name.rsplit('.', 1)
+  def ResolveNamespace(self, full_namespace):
     filenames = GenerateFilenames(full_namespace)
     for path, namespace in self._include_rules:
       for filename in reversed(filenames):
         filepath = os.path.join(path, filename);
         if os.path.exists(filepath):
           namespace = Model().AddNamespace(
               self.LoadSchema(filepath)[0],
               os.path.join(self._display_path, filepath),
               environment=CppNamespaceEnvironment(namespace))
-          if type_name in namespace.types:
-            return namespace
+          return namespace

[not at google - send to devlin, 2014/08/28 19:41:35]

(I know the code is already like this but...) |namespace| is assigned two
different concepts here, the namespace name, and the namespace model. The former
should probably be "cpp_namespace".

Also you could just return |namespace| immediately and not need to assign it to
a variable at all.

[lfg, 2014/08/28 23:14:59]

Done.

+    return None
+
+  def ResolveType(self, full_name, default_namespace):
+    name_parts = full_name.rsplit('.', 1)
+    if len(name_parts) == 1:
+      if full_name not in default_namespace.types:
+        return None
+      return default_namespace
+    full_namespace, type_name = full_name.rsplit('.', 1)
+    namespace = self.ResolveNamespace(full_namespace)
+    if namespace and type_name in namespace.types:
+      return namespace
     return None
 
   def LoadSchema(self, schema):
     '''Load a schema definition. The schema parameter must be a file name
-    without any path component - the file is loaded from the path defined by
-    the real_path argument passed to the constructor.'''
+    with the full path.'''
     schema_filename, schema_extension = os.path.splitext(schema)
 
-    schema_path = os.path.join(self._real_path, schema);
     if schema_extension == '.json':
-      api_defs = json_schema.Load(schema_path)
+      api_defs = json_schema.Load(schema)
     elif schema_extension == '.idl':
-      api_defs = idl_schema.Load(schema_path)
+      api_defs = idl_schema.Load(schema)
     else:
       sys.exit('Did not recognize file extension %s for schema %s' %
                (schema_extension, schema))
 
     return api_defs