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

Unified Diff: third_party/WebKit/Source/bindings/templates/methods.cpp

Issue 1883663005: Remove remaining binding layer RawPtr<>s. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/templates/methods.cpp
diff --git a/third_party/WebKit/Source/bindings/templates/methods.cpp b/third_party/WebKit/Source/bindings/templates/methods.cpp
index 3a9742f31d15a554a115c83b3074ef0082da891c..71753c39abc154eb35551bf6ec83db3eb29bb853 100644
--- a/third_party/WebKit/Source/bindings/templates/methods.cpp
+++ b/third_party/WebKit/Source/bindings/templates/methods.cpp
@@ -83,7 +83,7 @@ static void {{method.name}}{{method.overload_index}}Method{{world_suffix}}(const
{######################################}
{% macro generate_arguments(method, world_suffix) %}
{% for argument in method.arguments %}
-{{generate_argument_var_declaration(argument)}};
+{{argument.cpp_type}} {{argument.name}};
{% endfor %}
{
{% if method.has_optional_argument_without_default_value %}
@@ -113,17 +113,6 @@ static void {{method.name}}{{method.overload_index}}Method{{world_suffix}}(const
{######################################}
-{% macro generate_argument_var_declaration(argument) %}
-{# FIXME: remove EventListener special case #}
-{% if argument.idl_type == 'EventListener' %}
-RawPtr<{{argument.idl_type}}> {{argument.name}}
-{%- else %}
-{{argument.cpp_type}} {{argument.name}}
-{%- endif %}{# argument.idl_type == 'EventListener' #}
-{% endmacro %}
-
-
-{######################################}
{% macro generate_argument(method, argument, world_suffix) %}
{% if argument.is_optional_without_default_value %}
{# Optional arguments without a default value generate an early call with

Powered by Google App Engine
This is Rietveld 408576698