Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(520)

Unified Diff: sky/engine/bindings/scripts/idl_reader.py

Issue 922053002: Remove unused V8 integration code in Sky (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/bindings/scripts/idl_definitions.py ('k') | sky/engine/bindings/scripts/idl_types.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/bindings/scripts/idl_reader.py
diff --git a/sky/engine/bindings/scripts/idl_reader.py b/sky/engine/bindings/scripts/idl_reader.py
deleted file mode 100644
index 2f9c9f2acd0eacd63dcfd8f3f94b7e916d5c543c..0000000000000000000000000000000000000000
--- a/sky/engine/bindings/scripts/idl_reader.py
+++ /dev/null
@@ -1,111 +0,0 @@
-# Copyright (C) 2013 Google Inc. All rights reserved.
-#
-# Redistribution and use in source and binary forms, with or without
-# modification, are permitted provided that the following conditions are
-# met:
-#
-# * Redistributions of source code must retain the above copyright
-# notice, this list of conditions and the following disclaimer.
-# * Redistributions in binary form must reproduce the above
-# copyright notice, this list of conditions and the following disclaimer
-# in the documentation and/or other materials provided with the
-# distribution.
-# * Neither the name of Google Inc. nor the names of its
-# contributors may be used to endorse or promote products derived from
-# this software without specific prior written permission.
-#
-# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-"""Read an IDL file or complete IDL interface, producing an IdlDefinitions object.
-
-Design doc:
-http://www.chromium.org/developers/design-documents/idl-compiler#TOC-Front-end
-"""
-
-import os
-
-import blink_idl_parser
-from blink_idl_parser import BlinkIDLParser
-from idl_definitions import IdlDefinitions
-from idl_validator import EXTENDED_ATTRIBUTES_RELATIVE_PATH, IDLInvalidExtendedAttributeError, IDLExtendedAttributeValidator
-from interface_dependency_resolver import InterfaceDependencyResolver
-
-
-class IdlReader(object):
- def __init__(self, interfaces_info=None, outputdir=''):
- self.extended_attribute_validator = IDLExtendedAttributeValidator()
-
- if interfaces_info:
- self.interface_dependency_resolver = InterfaceDependencyResolver(interfaces_info, self)
- else:
- self.interface_dependency_resolver = None
-
- self.parser = BlinkIDLParser(outputdir=outputdir)
-
- def read_idl_definitions(self, idl_filename):
- """Returns an IdlDefinitions object for an IDL file, including all dependencies."""
- definitions = self.read_idl_file(idl_filename)
- if not self.interface_dependency_resolver:
- return definitions
- self.interface_dependency_resolver.resolve_dependencies(definitions)
- return definitions
-
- def read_idl_file(self, idl_filename):
- """Returns an IdlDefinitions object for an IDL file, without any dependencies.
-
- The IdlDefinitions object is guaranteed to contain a single
- IdlInterface; it may also contain other definitions, such as
- callback functions and enumerations."""
- ast = blink_idl_parser.parse_file(self.parser, idl_filename)
- if not ast:
- raise Exception('Failed to parse %s' % idl_filename)
- idl_file_basename, _ = os.path.splitext(os.path.basename(idl_filename))
- definitions = IdlDefinitions(idl_file_basename, ast)
-
- # Validate file contents with filename convention
- # The Blink IDL filenaming convention is that the file
- # <definition_name>.idl MUST contain exactly 1 definition
- # (interface, dictionary or exception), and the definition name must
- # agree with the file's basename, unless it is a partial definition.
- # (e.g., 'partial interface Foo' can be in FooBar.idl).
- targets = (definitions.interfaces.values() +
- definitions.dictionaries.values())
- number_of_targets = len(targets)
- if number_of_targets != 1:
- raise Exception(
- 'Expected exactly 1 definition in file {0}, but found {1}'
- .format(idl_filename, number_of_targets))
- target = targets[0]
- if not target.is_partial and target.name != idl_file_basename:
- raise Exception(
- 'Definition name "{0}" disagrees with IDL file basename "{1}".'
- .format(target.name, idl_file_basename))
-
- # Validate extended attributes
- if not self.extended_attribute_validator:
- return definitions
-
- try:
- self.extended_attribute_validator.validate_extended_attributes(definitions)
- except IDLInvalidExtendedAttributeError as error:
- raise IDLInvalidExtendedAttributeError("""
-IDL ATTRIBUTE ERROR in file:
-%s:
- %s
-If you want to add a new IDL extended attribute, please add it to:
- %s
-and add an explanation to the Blink IDL documentation at:
- http://www.chromium.org/blink/webidl/blink-idl-extended-attributes
- """ % (idl_filename, str(error), EXTENDED_ATTRIBUTES_RELATIVE_PATH))
-
- return definitions
« no previous file with comments | « sky/engine/bindings/scripts/idl_definitions.py ('k') | sky/engine/bindings/scripts/idl_types.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698