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

Side by Side Diff: third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl

Issue 2494973002: bindings: Stop using Nullable<T> in dictionary impl (Closed)
Patch Set: Comments Created 4 years, 1 month 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
OLDNEW
1 {% filter format_blink_cpp_source_code %} 1 {% filter format_blink_cpp_source_code %}
2 2
3 {% include 'copyright_block.txt' %} 3 {% include 'copyright_block.txt' %}
4 #ifndef {{cpp_class}}_h 4 #ifndef {{cpp_class}}_h
5 #define {{cpp_class}}_h 5 #define {{cpp_class}}_h
6 6
7 {% for filename in header_includes %} 7 {% for filename in header_includes %}
8 #include "{{filename}}" 8 #include "{{filename}}"
9 {% endfor %} 9 {% endfor %}
10 10
11 namespace blink { 11 namespace blink {
12 12
13 {# FIXME: Add "final" if this class doesn\'t have subclasses #} 13 {# FIXME: Add "final" if this class doesn\'t have subclasses #}
14 class {{exported}}{{cpp_class}}{% if parent_cpp_class %} : public {{parent_cpp_c lass}}{% endif %} { 14 class {{exported}}{{cpp_class}}{% if parent_cpp_class %} : public {{parent_cpp_c lass}}{% endif %} {
15 DISALLOW_NEW_EXCEPT_PLACEMENT_NEW(); 15 DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
16 public: 16 public:
17 {{cpp_class}}(); 17 {{cpp_class}}();
18 virtual ~{{cpp_class}}(); 18 virtual ~{{cpp_class}}();
19 19
20 {% for member in members %} 20 {% for member in members %}
21 bool {{member.has_method_name}}() const { return {{member.has_method_expressio n}}; } 21 bool {{member.has_method_name}}() const;
22 {{member.rvalue_cpp_type}} {{member.cpp_name}}() const { return {{member.gette r_expression}}; } 22 {{member.rvalue_cpp_type}} {{member.cpp_name}}() const;
23 void {{member.setter_name}}({{member.rvalue_cpp_type}} value) { m_{{member.cpp _name}} = value; } 23 void {{member.setter_name}}({{member.rvalue_cpp_type}});
24 {% if member.null_setter_name %} 24 {% if member.null_setter_name %}
25 void {{member.null_setter_name}}() { m_{{member.cpp_name}} = {{member.member_c pp_type}}(); } 25 void {{member.null_setter_name}}();
26 {% endif %} 26 {% endif %}
27 27
28 {% endfor %} 28 {% endfor %}
29 v8::Local<v8::Value> toV8Impl(v8::Local<v8::Object>, v8::Isolate*) const overr ide; 29 v8::Local<v8::Value> toV8Impl(v8::Local<v8::Object>, v8::Isolate*) const overr ide;
30 DECLARE_VIRTUAL_TRACE(); 30 DECLARE_VIRTUAL_TRACE();
31 31
32 private: 32 private:
33 {% for member in members %} 33 {% for member in members %}
34 {% if member.nullable_indicator_name %}
35 bool {{member.nullable_indicator_name}} = false;
36 {% endif %}
34 {{member.member_cpp_type}} m_{{member.cpp_name}}; 37 {{member.member_cpp_type}} m_{{member.cpp_name}};
35 {% endfor %} 38 {% endfor %}
36 39
37 friend class V8{{cpp_class}}; 40 friend class V8{{cpp_class}};
38 }; 41 };
39 42
40 } // namespace blink 43 } // namespace blink
41 44
42 #endif // {{cpp_class}}_h 45 #endif // {{cpp_class}}_h
43 46
44 {% endfilter %}{# format_blink_cpp_source_code #} 47 {% endfilter %}{# format_blink_cpp_source_code #}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698