I am referring to this recent checkin:

http://code.google.com/p/google-web-toolkit/source/diff?spec=svn6936&r=6936&format=side&path=/releases/2.0/user/src/com/google/gwt/json/client/JSONObject.java

The checkin comment states the change is needed for Chrome, but it is
checked into the main JSONObject, not into a ChromeJSONObject.  I
would expect it to be in a Chrome class because:

1. It would keep the generated code for non-Chrome browsers smaller.
2. The change itself slows down non-Chrome browsers unnecessarily.

JSONObject is a well-used object and it is important to keep it as
small and fast is humanly possible.

--

You received this message because you are subscribed to the Google Groups 
"Google Web Toolkit" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=.


Reply via email to