[Webkit-unassigned] [Bug 77295] V8 idl code generator doesn't handle [CachedAttribute] on SerializedScriptValue attributes.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=77295





--- Comment #14 from Pablo Flouret <pablof at motorola.com>  2012-02-02 16:34:01 PST ---
(From update of attachment 124611)
View in context: https://bugs.webkit.org/attachment.cgi?id=124611&action=review

>> Source/WebCore/bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:88
>> +    info.Holder()->DeleteHiddenValue(v8::String::NewSymbol("cachedValue")); // Invalidate the cached value.
> 
> Why don't you set the new value by SetHiddenValue(), instead of invalidating the current value?

We'd need to deserialize it, i figured it would be better to do that lazily when it's requested (i think the JSC generator does it like this as well). Also, (using this case as an example) we'd need to assume that v == imp->getCachedValue(), otherwise we'd need to call imp->getCachedValue() before deserializing, and by that point we might as well just let the getter do its job.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.