[Webkit-unassigned] [Bug 107207] Support selectively wrapping DOM accesses from certain V8 contexts.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 18 10:30:45 PST 2013


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





--- Comment #7 from Adam Barth <abarth at webkit.org>  2013-01-18 10:32:33 PST ---
(From update of attachment 183488)
View in context: https://bugs.webkit.org/attachment.cgi?id=183488&action=review

> Source/WebCore/bindings/v8/V8DOMCoverWrappingPolicy.cpp:67
> +    OwnPtr<WTF::String> m_logString;

There's no point in having an OwnPtr to a String.  You should just hold the String directly.  It's just a stack-allocated object that points to a reference counted object that holds the underlying string data.

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



More information about the webkit-unassigned mailing list