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

Unified Diff: Source/bindings/tests/results/V8TestObject.cpp

Issue 16836015: Rebaseline bindings test results after r152270 (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/bindings/tests/results/V8TestObject.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/tests/results/V8TestObject.cpp
diff --git a/Source/bindings/tests/results/V8TestObject.cpp b/Source/bindings/tests/results/V8TestObject.cpp
index 24a87e7acc7d8b580ba387a53ed1ba1b5270e65e..9da750987e04f08cdc0eba6320f165ccf5d5650a 100644
--- a/Source/bindings/tests/results/V8TestObject.cpp
+++ b/Source/bindings/tests/results/V8TestObject.cpp
@@ -4720,35 +4720,35 @@ void V8TestObject::constructorCallback(const v8::FunctionCallbackInfo<v8::Value>
TestObjV8Internal::constructor(args);
}
-v8::Handle<v8::Value> V8TestObject::indexedPropertyGetter(uint32_t index, const v8::AccessorInfo& info)
+void V8TestObject::indexedPropertyGetter(uint32_t index, const v8::PropertyCallbackInfo<v8::Value>& info)
{
ASSERT(V8DOMWrapper::maybeDOMWrapper(info.Holder()));
TestObj* collection = toNative(info.Holder());
RefPtr<Node> element = collection->item(index);
if (!element)
- return v8Undefined();
- return toV8Fast(element.release(), info, collection);
+ return;
+ v8SetReturnValue(info, toV8Fast(element.release(), info, collection));
}
-v8::Handle<v8::Value> V8TestObject::namedPropertyGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
+void V8TestObject::namedPropertyGetter(v8::Local<v8::String> name, const v8::PropertyCallbackInfo<v8::Value>& info)
{
if (!info.Holder()->GetRealNamedPropertyInPrototypeChain(name).IsEmpty())
- return v8Undefined();
+ return;
if (info.Holder()->HasRealNamedCallbackProperty(name))
- return v8Undefined();
+ return;
if (info.Holder()->HasRealNamedProperty(name))
- return v8Undefined();
+ return;
ASSERT(V8DOMWrapper::maybeDOMWrapper(info.Holder()));
TestObj* collection = toNative(info.Holder());
AtomicString propertyName = toWebCoreAtomicString(name);
String element = collection->namedItem(propertyName);
if (element.isNull())
- return v8Undefined();
- return v8String(element, info.GetIsolate());
+ return;
+ v8SetReturnValue(info, v8String(element, info.GetIsolate()));
}
-v8::Handle<v8::Array> V8TestObject::namedPropertyEnumerator(const v8::AccessorInfo& info)
+void V8TestObject::namedPropertyEnumerator(const v8::PropertyCallbackInfo<v8::Array>& info)
{
ExceptionCode ec = 0;
TestObj* collection = toNative(info.Holder());
@@ -4756,12 +4756,12 @@ v8::Handle<v8::Array> V8TestObject::namedPropertyEnumerator(const v8::AccessorIn
collection->namedPropertyEnumerator(names, ec);
if (ec) {
setDOMException(ec, info.GetIsolate());
- return v8::Handle<v8::Array>();
+ return;
}
v8::Handle<v8::Array> v8names = v8::Array::New(names.size());
for (size_t i = 0; i < names.size(); ++i)
v8names->Set(v8Integer(i, info.GetIsolate()), v8String(names[i], info.GetIsolate()));
- return v8names;
+ v8SetReturnValue(info, v8names);
}
static v8::Handle<v8::FunctionTemplate> ConfigureV8TestObjectTemplate(v8::Handle<v8::FunctionTemplate> desc, v8::Isolate* isolate, WrapperWorldType currentWorldType)
« no previous file with comments | « Source/bindings/tests/results/V8TestObject.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698