Revert revision 176015

tools/json_schema_compiler/model.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 copy
 import os.path
 import re
 
 from json_parse import OrderedDict
 
 class ParseException(Exception):
   """Thrown when data in the model is invalid.
   """
   def __init__(self, parent, message):
     hierarchy = _GetModelHierarchy(parent)
     hierarchy.append(message)
     Exception.__init__(
         self, 'Model parse exception at:\n' + '\n'.join(hierarchy))
 
 class Model(object):
   """Model of all namespaces that comprise an API.
 
   Properties:
   - |namespaces| a map of a namespace name to its model.Namespace
   """
   def __init__(self):
     self.namespaces = {}
 
-  def AddNamespace(self, json, source_file, include_compiler_options=False):
+  def AddNamespace(self, json, source_file):
     """Add a namespace's json to the model and returns the namespace.
     """
-    namespace = Namespace(json,
-                          source_file,
-                          include_compiler_options=include_compiler_options)
+    namespace = Namespace(json, source_file)
     self.namespaces[namespace.name] = namespace
     return namespace
 
 class Namespace(object):
   """An API namespace.
 
   Properties:
   - |name| the name of the namespace
   - |unix_name| the unix_name of the namespace
   - |source_file| the file that contained the namespace definition
   - |source_file_dir| the directory component of |source_file|
   - |source_file_filename| the filename component of |source_file|
-  - |platforms| if not None, the list of platforms that the namespace is
-                available to
   - |types| a map of type names to their model.Type
   - |functions| a map of function names to their model.Function
   - |events| a map of event names to their model.Function
   - |properties| a map of property names to their model.Property
-  - |compiler_options| the compiler_options dict, only present if
-                       |include_compiler_options| is True
   """
-  def __init__(self, json, source_file, include_compiler_options=False):
+  def __init__(self, json, source_file):
     self.name = json['namespace']
     self.unix_name = UnixName(self.name)
     self.source_file = source_file
     self.source_file_dir, self.source_file_filename = os.path.split(source_file)
     self.parent = None
-    self.platforms = _GetPlatforms(json)
     _AddTypes(self, json, self)
     _AddFunctions(self, json, self)
     _AddEvents(self, json, self)
     _AddProperties(self, json, self)
-    if include_compiler_options:
-      self.compiler_options = json.get('compiler_options', {})
 
 class Type(object):
   """A Type defined in the json.
 
   Properties:
   - |name| the type name
   - |description| the description of the type (if provided)
   - |properties| a map of property unix_names to their model.Property
   - |functions| a map of function names to their model.Function
   - |events| a map of event names to their model.Event
@@ skipping 50 matching lines @@
           additional_properties_key,
           additional_properties,
           namespace,
           is_additional_properties=True)
 
 class Function(object):
   """A Function defined in the API.
 
   Properties:
   - |name| the function name
-  - |platforms| if not None, the list of platforms that the function is
-                available to
   - |params| a list of parameters to the function (order matters). A separate
-             parameter is used for each choice of a 'choices' parameter
+    parameter is used for each choice of a 'choices' parameter.
   - |description| a description of the function (if provided)
   - |callback| the callback parameter to the function. There should be exactly
-               one
+    one
   - |optional| whether the Function is "optional"; this only makes sense to be
-               present when the Function is representing a callback property
+    present when the Function is representing a callback property.
   - |simple_name| the name of this Function without a namespace
   """
   def __init__(self,
                parent,
                json,
                namespace,
                from_json=False,
                from_client=False):
     self.name = json['name']
     self.simple_name = _StripNamespace(self.name, namespace)
-    self.platforms = _GetPlatforms(json)
     self.params = []
     self.description = json.get('description')
     self.callback = None
     self.optional = json.get('optional', False)
     self.parent = parent
     self.nocompile = json.get('nocompile')
     options = json.get('options', {})
     self.conditions = options.get('conditions', [])
     self.actions = options.get('actions', [])
     self.supports_listeners = options.get('supportsListeners', True)
@@ skipping 193 matching lines @@
   def Copy(self):
     """Makes a copy of this model.Property object and allow the unix_name to be
     set again.
     """
     property_copy = copy.copy(self)
     property_copy._unix_name_used = False
     return property_copy
 
   unix_name = property(GetUnixName, SetUnixName)
 
-class _Enum(object):
-  """Superclass for enum types with a "name" field, setting up repr/eq/ne.
-  Enums need to do this so that equality/non-equality work over pickling.
+class _PropertyTypeInfo(object):
+  """This class is not an inner class of |PropertyType| so it can be pickled.
   """
-
-  @staticmethod
-  def GetAll(cls):
-    """Yields all _Enum objects declared in |cls|.
-    """
-    for prop_key in dir(cls):
-      prop_value = getattr(cls, prop_key)
-      if isinstance(prop_value, _Enum):
-        yield prop_value
-
-  def __init__(self, name):
+  def __init__(self, is_fundamental, name):
+    self.is_fundamental = is_fundamental
     self.name = name
 
-  def __repr(self):
+  def __repr__(self):
     return self.name
 
   def __eq__(self, other):
-    return type(other) == type(self) and other.name == self.name
+    return isinstance(other, _PropertyTypeInfo) and self.name == other.name
 
   def __ne__(self, other):
+    # Yes. You seriously do need this.
     return not (self == other)
 
-class _PropertyTypeInfo(_Enum):
-  def __init__(self, is_fundamental, name):
-    _Enum.__init__(self, name)
-    self.is_fundamental = is_fundamental
-
 class PropertyType(object):
   """Enum of different types of properties/parameters.
   """
   INTEGER = _PropertyTypeInfo(True, "INTEGER")
   INT64 = _PropertyTypeInfo(True, "INT64")
   DOUBLE = _PropertyTypeInfo(True, "DOUBLE")
   BOOLEAN = _PropertyTypeInfo(True, "BOOLEAN")
   STRING = _PropertyTypeInfo(True, "STRING")
   ENUM = _PropertyTypeInfo(False, "ENUM")
   ARRAY = _PropertyTypeInfo(False, "ARRAY")
@@ skipping 67 matching lines @@
   """
   model.properties = OrderedDict()
   for name, property_json in json.get('properties', {}).items():
     model.properties[name] = Property(
         model,
         name,
         property_json,
         namespace,
         from_json=from_json,
         from_client=from_client)
-
-class _PlatformInfo(_Enum):
-  def __init__(self, name):
-    _Enum.__init__(self, name)
-
-class Platforms(object):
-  """Enum of the possible platforms.
-  """
-  CHROMEOS = _PlatformInfo("chromeos")
-  CHROMEOS_TOUCH = _PlatformInfo("chromeos_touch")
-  LINUX = _PlatformInfo("linux")
-  MAC = _PlatformInfo("mac")
-  WIN = _PlatformInfo("win")
-
-def _GetPlatforms(json):
-  if 'platforms' not in json:
-    return None
-  platforms = []
-  for platform_name in json['platforms']:
-    for platform_enum in _Enum.GetAll(Platforms):
-      if platform_name == platform_enum.name:
-        platforms.append(platform_enum)
-        break
-  return platforms