Fix up how the JSON Schema compiler decides whether to include or forward

declare references to C++ types in other files. This fixed up a problem where
references as part of required properties would cause compilation errors,
specifically for the use of tabs.InjectDetails from the webview API.


BUG=171726


Review URL: https://chromiumcodereview.appspot.com/11953121

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179232 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 8 deletions

diff --git a/tools/json_schema_compiler/schema_util.py b/tools/json_schema_compiler/schema_util.py
index e71be8b..7ce399e 100644
--- a/tools/json_schema_compiler/schema_util.py
+++ b/tools/json_schema_compiler/schema_util.py
@@ -9,15 +9,19 @@ import json_parse
 def CapitalizeFirstLetter(value):
   return value[0].capitalize() + value[1:]
 
-def GetNamespace(ref_type):
-  if '.' in ref_type:
-    return ref_type[:ref_type.rindex('.')]
+def GetNamespace(ref):
+  return SplitNamespace(ref)[0]
 
-def StripSchemaNamespace(s):
-  last_dot = s.rfind('.')
-  if not last_dot == -1:
-    return s[last_dot + 1:]
-  return s
+def StripNamespace(ref):
+  return SplitNamespace(ref)[1]
+
+def SplitNamespace(ref):
+  """Returns (namespace, entity) from |ref|, e.g. app.window.AppWindow ->
+  (app.window, AppWindow). If |ref| isn't qualified then returns (None, ref).
+  """
+  if '.' in ref:
+    return tuple(ref.rsplit('.', 1))
+  return (None, ref)
 
 def JsFunctionNameToClassName(namespace_name, function_name):
   """Transform a fully qualified function name like foo.bar.baz into FooBarBaz