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

Side by Side Diff: Source/bindings/tests/results/V8TestOverloadedConstructors.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 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 192
193 bool V8TestOverloadedConstructors::HasInstanceInAnyWorld(v8::Handle<v8::Value> v alue, v8::Isolate* isolate) 193 bool V8TestOverloadedConstructors::HasInstanceInAnyWorld(v8::Handle<v8::Value> v alue, v8::Isolate* isolate)
194 { 194 {
195 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld) 195 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld)
196 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld) 196 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld)
197 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d); 197 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d);
198 } 198 }
199 199
200 v8::Handle<v8::Object> V8TestOverloadedConstructors::createWrapper(PassRefPtr<Te stOverloadedConstructors> impl, v8::Handle<v8::Object> creationContext, v8::Isol ate* isolate) 200 v8::Handle<v8::Object> V8TestOverloadedConstructors::createWrapper(PassRefPtr<Te stOverloadedConstructors> impl, v8::Handle<v8::Object> creationContext, v8::Isol ate* isolate)
201 { 201 {
202 ASSERT(impl.get()); 202 ASSERT(impl);
203 ASSERT(!DOMDataStore::containsWrapper<V8TestOverloadedConstructors>(impl.get (), isolate)); 203 ASSERT(!DOMDataStore::containsWrapper<V8TestOverloadedConstructors>(impl.get (), isolate));
204 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 204 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
205 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 205 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
206 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 206 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
207 // the same object de-ref functions, though, so use that as the basis of the check. 207 // the same object de-ref functions, though, so use that as the basis of the check.
208 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 208 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
209 } 209 }
210 210
211 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 211 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
212 if (UNLIKELY(wrapper.IsEmpty())) 212 if (UNLIKELY(wrapper.IsEmpty()))
213 return wrapper; 213 return wrapper;
214 214
215 installPerContextProperties(wrapper, impl.get(), isolate); 215 installPerContextProperties(wrapper, impl.get(), isolate);
216 V8DOMWrapper::associateObjectWithWrapper<V8TestOverloadedConstructors>(impl, &info, wrapper, isolate, WrapperConfiguration::Independent); 216 V8DOMWrapper::associateObjectWithWrapper<V8TestOverloadedConstructors>(impl, &info, wrapper, isolate, WrapperConfiguration::Independent);
217 return wrapper; 217 return wrapper;
218 } 218 }
219 219
220 void V8TestOverloadedConstructors::derefObject(void* object) 220 void V8TestOverloadedConstructors::derefObject(void* object)
221 { 221 {
222 fromInternalPointer(object)->deref(); 222 fromInternalPointer(object)->deref();
223 } 223 }
224 224
225 } // namespace WebCore 225 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698