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

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

Issue 519703002: Add [NoImplHeader] extended attribute (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 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
Index: Source/bindings/scripts/compute_interfaces_info_overall.py
diff --git a/Source/bindings/scripts/compute_interfaces_info_overall.py b/Source/bindings/scripts/compute_interfaces_info_overall.py
index 06395137ff93f63bec80fb55fb9b8815f1d05dfa..51cee60dc8c4df7ee2847b916a0ba949480b451d 100755
--- a/Source/bindings/scripts/compute_interfaces_info_overall.py
+++ b/Source/bindings/scripts/compute_interfaces_info_overall.py
@@ -228,10 +228,11 @@ def compute_interfaces_info_overall(info_individuals):
# However, they are needed for legacy implemented interfaces that
# are being treated as partial interfaces, until we remove these.
# http://crbug.com/360435
- implemented_interfaces_include_paths = [
- implemented_interface_info['include_path']
- for implemented_interface_info in implemented_interfaces_info
- if implemented_interface_info['is_legacy_treat_as_partial_interface']]
+ implemented_interfaces_include_paths = []
+ for implemented_interface_info in implemented_interfaces_info:
+ if (implemented_interface_info['is_legacy_treat_as_partial_interface'] and
+ implemented_interface_info['include_path']):
+ implemented_interfaces_include_paths.append(implemented_interface_info['include_path'])
interface_info.update({
'dependencies_full_paths': (partial_interfaces_full_paths +
« no previous file with comments | « Source/bindings/scripts/compute_interfaces_info_individual.py ('k') | Source/core/testing/PartialPrivateScriptTest.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698