Chromium Code Reviews| Index: third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl |
| diff --git a/third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl b/third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl |
| index f1c95caa67809afd7943d234d617b89d6dbb6592..634910bded1901de56b61201e87020d81099dca7 100644 |
| --- a/third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl |
| +++ b/third_party/WebKit/Source/bindings/templates/interface.cpp.tmpl |
| @@ -841,7 +841,7 @@ v8::Local<v8::Object> {{v8_class}}::findInstanceInPrototypeChain(v8::Local<v8::V |
| WTF::ArrayBufferContents contents(v8Contents.Data(), v8Contents.ByteLength(), WTF::ArrayBufferContents::{% if interface_name == 'ArrayBuffer' %}Not{% endif %}Shared); |
| {{cpp_class}}* buffer = {{cpp_class}}::create(contents); |
| v8::Local<v8::Object> associatedWrapper = buffer->associateWithWrapper(v8::Isolate::GetCurrent(), buffer->wrapperTypeInfo(), object); |
| - ASSERT_UNUSED(associatedWrapper, associatedWrapper == object); |
| + DCHECK(associatedWrapper == object); |
|
Yuta Kitamura
2016/10/06 10:50:04
Does DCHECK_EQ cause some problem?
tkent
2016/10/06 14:07:19
DCHECK_EQ requires operator<< for v8::Local<>, and
|
| return buffer; |
| } |
| @@ -898,7 +898,7 @@ v8::Local<v8::Object> {{v8_class}}::findInstanceInPrototypeChain(v8::Local<v8::V |
| ASSERT_NOT_REACHED(); |
| } |
| v8::Local<v8::Object> associatedWrapper = typedArray->associateWithWrapper(v8::Isolate::GetCurrent(), typedArray->wrapperTypeInfo(), object); |
| - ASSERT_UNUSED(associatedWrapper, associatedWrapper == object); |
| + DCHECK(associatedWrapper == object); |
| return typedArray->toImpl<{{cpp_class}}>(); |
| } |