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

Unified Diff: Source/bindings/scripts/v8_utilities.py

Issue 1258863003: bindings: Moves [OverrideBuiltins]'s attributes to the prototype chain. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated the test results. Created 5 years, 5 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 | « LayoutTests/webexposed/global-interface-listing-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/scripts/v8_utilities.py
diff --git a/Source/bindings/scripts/v8_utilities.py b/Source/bindings/scripts/v8_utilities.py
index e1e584be75bec46fea220ae4828f19f69518b38c..149352085eb788a6162cc241b553ed45e5c7c454 100644
--- a/Source/bindings/scripts/v8_utilities.py
+++ b/Source/bindings/scripts/v8_utilities.py
@@ -461,8 +461,7 @@ def on_prototype(interface, member):
# These members must not be placed on prototype chains.
if (is_constructor_attribute(member) or
member.is_static or
- is_unforgeable(interface, member) or
- 'OverrideBuiltins' in interface.extended_attributes):
+ is_unforgeable(interface, member)):
return False
# TODO(yukishiino): We should handle [Global] and [PrimaryGlobal] instead of
« no previous file with comments | « LayoutTests/webexposed/global-interface-listing-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698