Chromium Code Reviews| Index: Source/bindings/templates/interface.cpp |
| diff --git a/Source/bindings/templates/interface.cpp b/Source/bindings/templates/interface.cpp |
| index 5da47a89b6533626e3f90a0aa2fd535dfe2c64cc..ce5dce512376cff0c350e05bf81921e48802faf9 100644 |
| --- a/Source/bindings/templates/interface.cpp |
| +++ b/Source/bindings/templates/interface.cpp |
| @@ -596,8 +596,7 @@ void {{v8_class}}::visitDOMWrapper(v8::Isolate* isolate, ScriptWrappable* script |
| {% for set_wrapper_reference_to in set_wrapper_reference_to_list %} |
| {{set_wrapper_reference_to.cpp_type}} {{set_wrapper_reference_to.name}} = impl->{{set_wrapper_reference_to.name}}(); |
| if ({{set_wrapper_reference_to.name}}) { |
| - if (!DOMDataStore::containsWrapper({{set_wrapper_reference_to.name}}, isolate)) |
| - {{set_wrapper_reference_to.name}}->wrap(creationContext, isolate); |
|
haraken
2015/02/16 03:56:49
Another idea would be just to skip creating a wrap
kouhei (in TOK)
2015/02/16 04:03:24
Wouldn't it be too fragile? I prefer an ASSERT her
|
| + ASSERT(DOMDataStore::containsWrapper({{set_wrapper_reference_to.name}}, isolate)); |
| DOMDataStore::setWrapperReference(wrapper, {{set_wrapper_reference_to.name}}, isolate); |
| } |
| {% endfor %} |