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

Side by Side Diff: Source/bindings/tests/results/V8Float64Array.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 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 196
197 bool V8Float64Array::HasInstanceInAnyWorld(v8::Handle<v8::Value> value, v8::Isol ate* isolate) 197 bool V8Float64Array::HasInstanceInAnyWorld(v8::Handle<v8::Value> value, v8::Isol ate* isolate)
198 { 198 {
199 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld) 199 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld)
200 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld) 200 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld)
201 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d); 201 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d);
202 } 202 }
203 203
204 v8::Handle<v8::Object> V8Float64Array::createWrapper(PassRefPtr<Float64Array> im pl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate) 204 v8::Handle<v8::Object> V8Float64Array::createWrapper(PassRefPtr<Float64Array> im pl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
205 { 205 {
206 ASSERT(impl.get()); 206 ASSERT(impl);
207 ASSERT(!DOMDataStore::containsWrapper<V8Float64Array>(impl.get(), isolate)); 207 ASSERT(!DOMDataStore::containsWrapper<V8Float64Array>(impl.get(), isolate));
208 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 208 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
209 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 209 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
210 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 210 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
211 // the same object de-ref functions, though, so use that as the basis of the check. 211 // the same object de-ref functions, though, so use that as the basis of the check.
212 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 212 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
213 } 213 }
214 214
215 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 215 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
216 if (UNLIKELY(wrapper.IsEmpty())) 216 if (UNLIKELY(wrapper.IsEmpty()))
217 return wrapper; 217 return wrapper;
218 218
219 impl->buffer()->setDeallocationObserver(V8ArrayBufferDeallocationObserver::i nstance()); 219 impl->buffer()->setDeallocationObserver(V8ArrayBufferDeallocationObserver::i nstance());
220 installPerContextProperties(wrapper, impl.get(), isolate); 220 installPerContextProperties(wrapper, impl.get(), isolate);
221 V8DOMWrapper::associateObjectWithWrapper<V8Float64Array>(impl, &info, wrappe r, isolate, WrapperConfiguration::Independent); 221 V8DOMWrapper::associateObjectWithWrapper<V8Float64Array>(impl, &info, wrappe r, isolate, WrapperConfiguration::Independent);
222 return wrapper; 222 return wrapper;
223 } 223 }
224 224
225 void V8Float64Array::derefObject(void* object) 225 void V8Float64Array::derefObject(void* object)
226 { 226 {
227 fromInternalPointer(object)->deref(); 227 fromInternalPointer(object)->deref();
228 } 228 }
229 229
230 } // namespace WebCore 230 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/bindings/scripts/code_generator_v8.pm ('k') | Source/bindings/tests/results/V8TestActiveDOMObject.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698