Revision: 9868
Author:   [email protected]
Date:     Sat Mar 19 13:11:29 2011
Log: Handle cases where element.canPlayType() returns 'no' instead of the empty string on some browsers.

Review at http://gwt-code-reviews.appspot.com/1362802

Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=9868

Modified:
 /trunk/user/src/com/google/gwt/dom/client/MediaElement.java

=======================================
--- /trunk/user/src/com/google/gwt/dom/client/MediaElement.java Fri Jan 28 12:43:16 2011 +++ /trunk/user/src/com/google/gwt/dom/client/MediaElement.java Sat Mar 19 13:11:29 2011
@@ -112,7 +112,11 @@
    *         {@link #CANNOT_PLAY}
    */
   public final native String canPlayType(String type) /*-{
-    return this.canPlayType(type);
+    var canPlayType = this.canPlayType(type);
+     // Some browsers report "no" instead of the empty string.
+     // See http://gwt-voices.appspot.com/
+    return canPlayType == "no" ?
+        @com.google.gwt.dom.client.MediaElement::CANNOT_PLAY : canPlayType;
   }-*/;

   /**

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

Reply via email to