[Webkit-unassigned] [Bug 88451] Extra HandleScope in V8Proxy

[bugzilla-daemon at webkit.org]

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


Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #146372|review?                     |review+
               Flag|                            |




--- Comment #18 from Adam Barth <abarth at webkit.org>  2012-06-13 10:00:05 PST ---
(From update of attachment 146372)
View in context: https://bugs.webkit.org/attachment.cgi?id=146372&action=review

> Source/WebCore/bindings/v8/V8Proxy.cpp:263
> +        if (evaluationResult.IsEmpty())
> +            evaluationResult = v8::Local<v8::Value>::New(v8::Undefined());

Does this line make sense?  I thought empty handles and v8::Undefined where the same...

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