John, your name was around in some of the these files, but feel free to
nominate someone else to review.


http://gwt-code-reviews.appspot.com/1725808/diff/1/user/src/com/google/gwt/dom/client/DOMImplStandardBase.java
File user/src/com/google/gwt/dom/client/DOMImplStandardBase.java (left):

http://gwt-code-reviews.appspot.com/1725808/diff/1/user/src/com/google/gwt/dom/client/DOMImplStandardBase.java#oldcode258
user/src/com/google/gwt/dom/client/DOMImplStandardBase.java:258: return
false;
Per a gwt-contrib thread that points to the commit that introduced this
hack, Node.contains was broken in Safari2. I haven't found the original
bug, but a patch against jquery asserts Safari3's Node.contains is fine.

http://gwt-code-reviews.appspot.com/1725808/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to