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

Side by Side Diff: lib/dom/scripts/systemnative.py

Issue 10224002: Remove now unnecessary custom getter suppressions. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 2 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
3 # for details. All rights reserved. Use of this source code is governed by a 3 # for details. All rights reserved. Use of this source code is governed by a
4 # BSD-style license that can be found in the LICENSE file. 4 # BSD-style license that can be found in the LICENSE file.
5 5
6 """This module provides shared functionality for the systems to generate 6 """This module provides shared functionality for the systems to generate
7 native binding from the IDL database.""" 7 native binding from the IDL database."""
8 8
9 import emitter 9 import emitter
10 import os 10 import os
(...skipping 446 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 457
458 return False 458 return False
459 459
460 def AddAttribute(self, getter, setter): 460 def AddAttribute(self, getter, setter):
461 if 'CheckSecurityForNode' in (getter or setter).ext_attrs: 461 if 'CheckSecurityForNode' in (getter or setter).ext_attrs:
462 # FIXME: exclude from interface as well. 462 # FIXME: exclude from interface as well.
463 return 463 return
464 464
465 # FIXME: these should go away. 465 # FIXME: these should go away.
466 classes_with_unsupported_custom_getters = [ 466 classes_with_unsupported_custom_getters = [
467 'Clipboard', 'Console', 'Coordinates', 'DeviceMotionEvent', 467 'Coordinates',
468 'DeviceOrientationEvent', 'FileReader', 'JavaScriptCallFrame', 468 'FileReader',
469 'HTMLInputElement', 'HTMLOptionsCollection', 'HTMLOutputElement', 469 'HTMLOutputElement',
470 'ScriptProfileNode', 'WebKitAnimation'] 470 'ScriptProfileNode',
471 'WebKitAnimation' ]
471 if (self._interface.id in classes_with_unsupported_custom_getters and 472 if (self._interface.id in classes_with_unsupported_custom_getters and
472 getter and set(['Custom', 'CustomGetter']) & set(getter.ext_attrs)): 473 getter and set(['Custom', 'CustomGetter']) & set(getter.ext_attrs)):
473 return 474 return
474 475
475 if getter: 476 if getter:
476 self._AddGetter(getter) 477 self._AddGetter(getter)
477 if setter: 478 if setter:
478 self._AddSetter(setter) 479 self._AddSetter(setter)
479 480
480 def _AddGetter(self, attr): 481 def _AddGetter(self, attr):
(...skipping 365 matching lines...) Expand 10 before | Expand all | Expand 10 after
846 def _InstanceOfNode(database, interface): 847 def _InstanceOfNode(database, interface):
847 if interface.id == 'Node': 848 if interface.id == 'Node':
848 return True 849 return True
849 for parent in interface.parents: 850 for parent in interface.parents:
850 if not database.HasInterface(parent.type.id): 851 if not database.HasInterface(parent.type.id):
851 continue 852 continue
852 parent_interface = database.GetInterface(parent.type.id) 853 parent_interface = database.GetInterface(parent.type.id)
853 if _InstanceOfNode(database, parent_interface): 854 if _InstanceOfNode(database, parent_interface):
854 return True 855 return True
855 return False 856 return False
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698