[Webkit-unassigned] [Bug 55343] Global JSON and Math should be configurable but aren't

[bugzilla-daemon at webkit.org]

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


Gavin Barraclough <barraclough at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #120897|                            |review?
               Flag|                            |




--- Comment #2 from Gavin Barraclough <barraclough at apple.com>  2012-01-02 15:41:25 PST ---
Created an attachment (id=120897)
 --> (https://bugs.webkit.org/attachment.cgi?id=120897&action=review)
Fix for JSON object

Our current handling of Math & JSON (treating them as non-configurable vars) allows slightly faster access.  This is probably not significant for typical JSON loads, since in real world usage the overhead of one property access from the global object is nothing compared to the cost of loading JSON data over the network, the host call to the parse function, and running the parser over the JSON data.  Fast access to the Math object is more important since it is used for very small, fast intrinsic functions that will be inlined by the JIT, e.g. abs.  We should fix this bug for the Math object too, but there we will need to pay more concern to avoiding any performance penalty – in the case of the JSON object there is no real justification to retain our current incorrect behavior.

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