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

Side by Side Diff: Source/bindings/tests/results/V8TestInterface.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 1121 matching lines...) Expand 10 before | Expand all | Expand 10 after
1132 #endif // ENABLE(Condition11) || ENABLE(Condition12) 1132 #endif // ENABLE(Condition11) || ENABLE(Condition12)
1133 } 1133 }
1134 1134
1135 ActiveDOMObject* V8TestInterface::toActiveDOMObject(v8::Handle<v8::Object> objec t) 1135 ActiveDOMObject* V8TestInterface::toActiveDOMObject(v8::Handle<v8::Object> objec t)
1136 { 1136 {
1137 return toNative(object); 1137 return toNative(object);
1138 } 1138 }
1139 1139
1140 v8::Handle<v8::Object> V8TestInterface::createWrapper(PassRefPtr<TestInterface> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate) 1140 v8::Handle<v8::Object> V8TestInterface::createWrapper(PassRefPtr<TestInterface> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
1141 { 1141 {
1142 ASSERT(impl.get()); 1142 ASSERT(impl);
1143 ASSERT(!DOMDataStore::containsWrapper<V8TestInterface>(impl.get(), isolate)) ; 1143 ASSERT(!DOMDataStore::containsWrapper<V8TestInterface>(impl.get(), isolate)) ;
1144 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) { 1144 if (ScriptWrappable::wrapperCanBeStoredInObject(impl.get())) {
1145 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get()); 1145 const WrapperTypeInfo* actualInfo = ScriptWrappable::getTypeInfoFromObje ct(impl.get());
1146 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have 1146 // Might be a XXXConstructor::info instead of an XXX::info. These will b oth have
1147 // the same object de-ref functions, though, so use that as the basis of the check. 1147 // the same object de-ref functions, though, so use that as the basis of the check.
1148 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction); 1148 RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(actualInfo->derefObjectFunction == info.derefObjectFunction);
1149 } 1149 }
1150 1150
1151 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate); 1151 v8::Handle<v8::Object> wrapper = V8DOMWrapper::createWrapper(creationContext , &info, toInternalPointer(impl.get()), isolate);
1152 if (UNLIKELY(wrapper.IsEmpty())) 1152 if (UNLIKELY(wrapper.IsEmpty()))
1153 return wrapper; 1153 return wrapper;
1154 1154
1155 installPerContextProperties(wrapper, impl.get(), isolate); 1155 installPerContextProperties(wrapper, impl.get(), isolate);
1156 V8DOMWrapper::associateObjectWithWrapper<V8TestInterface>(impl, &info, wrapp er, isolate, WrapperConfiguration::Dependent); 1156 V8DOMWrapper::associateObjectWithWrapper<V8TestInterface>(impl, &info, wrapp er, isolate, WrapperConfiguration::Dependent);
1157 return wrapper; 1157 return wrapper;
1158 } 1158 }
1159 1159
1160 void V8TestInterface::derefObject(void* object) 1160 void V8TestInterface::derefObject(void* object)
1161 { 1161 {
1162 fromInternalPointer(object)->deref(); 1162 fromInternalPointer(object)->deref();
1163 } 1163 }
1164 1164
1165 } // namespace WebCore 1165 } // namespace WebCore
1166 1166
1167 #endif // ENABLE(Condition1) || ENABLE(Condition2) 1167 #endif // ENABLE(Condition1) || ENABLE(Condition2)
OLDNEW
« no previous file with comments | « Source/bindings/tests/results/V8TestExtendedEvent.cpp ('k') | Source/bindings/tests/results/V8TestInterfaceEmpty.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698