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

Side by Side Diff: Source/bindings/tests/results/V8TestException.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 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 115
116 bool V8TestException::HasInstanceInAnyWorld(v8::Handle<v8::Value> value, v8::Iso late* isolate) 116 bool V8TestException::HasInstanceInAnyWorld(v8::Handle<v8::Value> value, v8::Iso late* isolate)
117 { 117 {
118 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld) 118 return V8PerIsolateData::from(isolate)->hasInstance(&info, value, MainWorld)
119 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld) 119 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, IsolatedWo rld)
120 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d); 120 || V8PerIsolateData::from(isolate)->hasInstance(&info, value, WorkerWorl d);
121 } 121 }
122 122
123 v8::Handle<v8::Object> V8TestException::createWrapper(PassRefPtr<TestException> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate) 123 v8::Handle<v8::Object> V8TestException::createWrapper(PassRefPtr<TestException> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
124 { 124 {
125 ASSERT(impl.get()); 125 ASSERT(impl);
126 ASSERT(!DOMDataStore::containsWrapper<V8TestException>(impl.get(), isolate)) ; 126 ASSERT(!DOMDataStore::containsWrapper<V8TestException>(impl.get(), isolate)) ;
127 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 127 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
128 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 128 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
129 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 129 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
130 // the same object de-ref functions, though, so use that as the basis of the check. 130 // the same object de-ref functions, though, so use that as the basis of the check.
131 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 131 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
132 } 132 }
133 133
134 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 134 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
135 if (UNLIKELY(wrapper.IsEmpty())) 135 if (UNLIKELY(wrapper.IsEmpty()))
136 return wrapper; 136 return wrapper;
137 137
138 installPerContextProperties(wrapper, impl.get(), isolate); 138 installPerContextProperties(wrapper, impl.get(), isolate);
139 V8DOMWrapper::associateObjectWithWrapper<V8TestException>(impl, &info, wrapp er, isolate, WrapperConfiguration::Independent); 139 V8DOMWrapper::associateObjectWithWrapper<V8TestException>(impl, &info, wrapp er, isolate, WrapperConfiguration::Independent);
140 return wrapper; 140 return wrapper;
141 } 141 }
142 142
143 void V8TestException::derefObject(void* object) 143 void V8TestException::derefObject(void* object)
144 { 144 {
145 fromInternalPointer(object)->deref(); 145 fromInternalPointer(object)->deref();
146 } 146 }
147 147
148 } // namespace WebCore 148 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/bindings/tests/results/V8TestEventTarget.cpp ('k') | Source/bindings/tests/results/V8TestExtendedEvent.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698