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

Side by Side Diff: Source/bindings/tests/results/V8TestNamedConstructor.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 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d); 168 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d);
169 } 169 }
170 170
171 ActiveDOMObject* V8TestNamedConstructor::toActiveDOMObject(v8::Handle<v8::Object > object) 171 ActiveDOMObject* V8TestNamedConstructor::toActiveDOMObject(v8::Handle<v8::Object > object)
172 { 172 {
173 return toNative(object); 173 return toNative(object);
174 } 174 }
175 175
176 v8::Handle<v8::Object> V8TestNamedConstructor::createWrapper(PassRefPtr<TestName dConstructor> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate ) 176 v8::Handle<v8::Object> V8TestNamedConstructor::createWrapper(PassRefPtr<TestName dConstructor> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate )
177 { 177 {
178 ASSERT(impl.get()); 178 ASSERT(impl);
179 ASSERT(!DOMDataStore::containsWrapper<V8TestNamedConstructor>(impl.get(), is olate)); 179 ASSERT(!DOMDataStore::containsWrapper<V8TestNamedConstructor>(impl.get(), is olate));
180 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 180 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
181 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 181 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
182 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 182 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
183 // the same object de-ref functions, though, so use that as the basis of the check. 183 // the same object de-ref functions, though, so use that as the basis of the check.
184 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 184 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
185 } 185 }
186 186
187 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 187 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
188 if (UNLIKELY(wrapper.IsEmpty())) 188 if (UNLIKELY(wrapper.IsEmpty()))
189 return wrapper; 189 return wrapper;
190 190
191 installPerContextProperties(wrapper, impl.get(), isolate); 191 installPerContextProperties(wrapper, impl.get(), isolate);
192 V8DOMWrapper::associateObjectWithWrapper<V8TestNamedConstructor>(impl, &info , wrapper, isolate, WrapperConfiguration::Dependent); 192 V8DOMWrapper::associateObjectWithWrapper<V8TestNamedConstructor>(impl, &info , wrapper, isolate, WrapperConfiguration::Dependent);
193 return wrapper; 193 return wrapper;
194 } 194 }
195 195
196 void V8TestNamedConstructor::derefObject(void* object) 196 void V8TestNamedConstructor::derefObject(void* object)
197 { 197 {
198 fromInternalPointer(object)->deref(); 198 fromInternalPointer(object)->deref();
199 } 199 }
200 200
201 } // namespace WebCore 201 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/bindings/tests/results/V8TestMediaQueryListListener.cpp ('k') | Source/bindings/tests/results/V8TestNode.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698