[Webkit-unassigned] [Bug 82442] Consider removal of removes jsNull()/v8::Null() from JSInternalsCustom.cpp/V8InternalsCustom.cpp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 28 10:33:47 PDT 2012


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





--- Comment #6 from Vineet Chaudhary (vineetc) <rgf748 at motorola.com>  2012-03-28 10:33:47 PST ---
(In reply to comment #5)
> (In reply to comment #2)
> > eric.carlson: Would you take a look at the change? (See also https://bugs.webkit.org/show_bug.cgi?id=82319) The behavior when the Vector is empty is not spec-ed, and we guess that we can remove the code that returns null when the Vector is empty.
> 
> This seems like a reasonable change to me.

As per comment #4 the test case covers empty vector case. Do I need to add more coverage for the test?

-- 
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