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

Side by Side Diff: Source/bindings/tests/results/V8TestEventTarget.cpp

Issue 26592006: Avoid useless call to RefPtr::get() in the generated bindings code (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 2 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 This file is part of the Blink open source project. 2 This file is part of the Blink open source project.
3 This file has been auto-generated by CodeGeneratorV8.pm. DO NOT MODIFY! 3 This file has been auto-generated by CodeGeneratorV8.pm. DO NOT MODIFY!
4 4
5 This library is free software; you can redistribute it and/or 5 This library is free software; you can redistribute it and/or
6 modify it under the terms of the GNU Library General Public 6 modify it under the terms of the GNU Library General Public
7 License as published by the Free Software Foundation; either 7 License as published by the Free Software Foundation; either
8 version 2 of the License, or (at your option) any later version. 8 version 2 of the License, or (at your option) any later version.
9 9
10 This library is distributed in the hope that it will be useful, 10 This library is distributed in the hope that it will be useful,
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d); 319 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d);
320 } 320 }
321 321
322 EventTarget* V8TestEventTarget::toEventTarget(v8::Handle<v8::Object> object) 322 EventTarget* V8TestEventTarget::toEventTarget(v8::Handle<v8::Object> object)
323 { 323 {
324 return toNative(object); 324 return toNative(object);
325 } 325 }
326 326
327 v8::Handle<v8::Object> V8TestEventTarget::createWrapper(PassRefPtr<TestEventTarg et> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate) 327 v8::Handle<v8::Object> V8TestEventTarget::createWrapper(PassRefPtr<TestEventTarg et> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
328 { 328 {
329 ASSERT(impl.get()); 329 ASSERT(impl);
330 ASSERT(!DOMDataStore::containsWrapper<V8TestEventTarget>(impl.get(), isolate )); 330 ASSERT(!DOMDataStore::containsWrapper<V8TestEventTarget>(impl.get(), isolate ));
331 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 331 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
332 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 332 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
333 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 333 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
334 // the same object de-ref functions, though, so use that as the basis of the check. 334 // the same object de-ref functions, though, so use that as the basis of the check.
335 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 335 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
336 } 336 }
337 337
338 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 338 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
339 if (UNLIKELY(wrapper.IsEmpty())) 339 if (UNLIKELY(wrapper.IsEmpty()))
340 return wrapper; 340 return wrapper;
341 341
342 installPerContextProperties(wrapper, impl.get(), isolate); 342 installPerContextProperties(wrapper, impl.get(), isolate);
343 V8DOMWrapper::associateObjectWithWrapper<V8TestEventTarget>(impl, &info, wra pper, isolate, WrapperConfiguration::Independent); 343 V8DOMWrapper::associateObjectWithWrapper<V8TestEventTarget>(impl, &info, wra pper, isolate, WrapperConfiguration::Independent);
344 return wrapper; 344 return wrapper;
345 } 345 }
346 346
347 void V8TestEventTarget::derefObject(void* object) 347 void V8TestEventTarget::derefObject(void* object)
348 { 348 {
349 fromInternalPointer(object)->deref(); 349 fromInternalPointer(object)->deref();
350 } 350 }
351 351
352 } // namespace WebCore 352 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/bindings/tests/results/V8TestEventConstructor.cpp ('k') | Source/bindings/tests/results/V8TestException.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698