Source/bindings/tests/results/modules/V8TestInterface5.cpp - Issue 1153613007: bindings: Use CreateDataProperty() instead of ForceSet()

Unified Diff: Source/bindings/tests/results/modules/V8TestInterface5.cpp

Issue 1153613007: bindings: Use CreateDataProperty() instead of ForceSet() (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Fix uninitialized error Created 5 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/core/V8TestTypedefs.cpp ('k') | Source/bindings/tests/results/modules/V8TestInterfacePartial.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: Source/bindings/tests/results/modules/V8TestInterface5.cpp

diff --git a/Source/bindings/tests/results/modules/V8TestInterface5.cpp b/Source/bindings/tests/results/modules/V8TestInterface5.cpp

index 877e81a67b6b5866d3b39b6cd6af2b18c0dfb5e9..abd0ea6e4a3960cba19bf35a4c06ef1014e65073 100644

--- a/Source/bindings/tests/results/modules/V8TestInterface5.cpp

+++ b/Source/bindings/tests/results/modules/V8TestInterface5.cpp

@@ -36,10 +36,10 @@ const WrapperTypeInfo& TestInterface5Implementation::s_wrapperTypeInfo = V8TestI

namespace TestInterface5ImplementationV8Internal {

template<class CallbackInfo>

-static void TestInterface5ImplementationForceSetAttributeOnThis(v8::Local<v8::Name> name, v8::Local<v8::Value> v8Value, const CallbackInfo& info)

+static bool TestInterface5ImplementationCreateDataProperty(v8::Local<v8::Name> name, v8::Local<v8::Value> v8Value, const CallbackInfo& info)

{

ASSERT(info.This()->IsObject());

- v8::Local<v8::Object>::Cast(info.This())->ForceSet(info.GetIsolate()->GetCurrentContext(), name, v8Value);

+ return v8CallBoolean(v8::Local<v8::Object>::Cast(info.This())->CreateDataProperty(info.GetIsolate()->GetCurrentContext(), name, v8Value));

}

static void TestInterface5ImplementationConstructorAttributeSetterCallback(v8::Local<v8::Name>, v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)

@@ -54,7 +54,7 @@ static void TestInterface5ImplementationConstructorAttributeSetterCallback(v8::L

const WrapperTypeInfo* wrapperTypeInfo = WrapperTypeInfo::unwrap(data);

if (!wrapperTypeInfo)

break;

- TestInterface5ImplementationForceSetAttributeOnThis(v8String(info.GetIsolate(), wrapperTypeInfo->interfaceName), v8Value, info);

+ TestInterface5ImplementationCreateDataProperty(v8String(info.GetIsolate(), wrapperTypeInfo->interfaceName), v8Value, info);

} while (false); // do ... while (false) just for use of break

TRACE_EVENT_SET_SAMPLING_STATE("v8", "V8Execution");

}

@@ -98,7 +98,7 @@ static void testInterfaceAttributeAttributeSetterCallback(const v8::FunctionCall

static void testInterfaceConstructorAttributeAttributeSetter(v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)

{

v8::Local<v8::String> propertyName = v8AtomicString(info.GetIsolate(), "testInterfaceConstructorAttribute");

- TestInterface5ImplementationForceSetAttributeOnThis(propertyName, v8Value, info);

+ TestInterface5ImplementationCreateDataProperty(propertyName, v8Value, info);

}

static void testInterfaceConstructorAttributeAttributeSetterCallback(v8::Local<v8::Name>, v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)