Title: [96891] trunk/LayoutTests
Revision
96891
Author
jer.no...@apple.com
Date
2011-10-06 19:33:23 -0700 (Thu, 06 Oct 2011)

Log Message

Some media tests fail when run inside a directory path containing periods.
https://bugs.webkit.org/show_bug.cgi?id=69580

Reviewed by Darin Adler.

To determine the extension, retrieve the last string following a period instead of the second.

* media/media-blocked-by-beforeload.html:
* media/video-source-load.html:
* media/video-source-moved.html:
* media/video-source-removed.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (96890 => 96891)


--- trunk/LayoutTests/ChangeLog	2011-10-07 02:23:03 UTC (rev 96890)
+++ trunk/LayoutTests/ChangeLog	2011-10-07 02:33:23 UTC (rev 96891)
@@ -1,3 +1,17 @@
+2011-10-06  Jer Noble  <jer.no...@apple.com>
+
+        Some media tests fail when run inside a directory path containing periods.
+        https://bugs.webkit.org/show_bug.cgi?id=69580
+
+        Reviewed by Darin Adler.
+
+        To determine the extension, retrieve the last string following a period instead of the second.
+
+        * media/media-blocked-by-beforeload.html:
+        * media/video-source-load.html:
+        * media/video-source-moved.html:
+        * media/video-source-removed.html:
+
 2011-10-06  Ryosuke Niwa  <rn...@webkit.org>
 
         Chromium Windows and Chromium Linux rebaselines for r96870. Also skip editing/pasteboard/smart-paste-008.html

Modified: trunk/LayoutTests/media/media-blocked-by-beforeload.html (96890 => 96891)


--- trunk/LayoutTests/media/media-blocked-by-beforeload.html	2011-10-07 02:23:03 UTC (rev 96890)
+++ trunk/LayoutTests/media/media-blocked-by-beforeload.html	2011-10-07 02:33:23 UTC (rev 96891)
@@ -43,12 +43,12 @@
 
             var blockedSource = document.createElement('source');
             blockedSource.src = ""
-            blockedSource.type = mimeTypeForExtension(blockedSource.src.split('.')[1]);
+            blockedSource.type = mimeTypeForExtension(blockedSource.src.split('.').pop());
             video.appendChild(blockedSource);
 
             var allowedSource = document.createElement('source');
             allowedSource.src = ""
-            allowedSource.type = mimeTypeForExtension(allowedSource.src.split('.')[1]);
+            allowedSource.type = mimeTypeForExtension(allowedSource.src.split('.').pop());
             video.appendChild(allowedSource);
 
             test = 2;

Modified: trunk/LayoutTests/media/video-source-load.html (96890 => 96891)


--- trunk/LayoutTests/media/video-source-load.html	2011-10-07 02:23:03 UTC (rev 96890)
+++ trunk/LayoutTests/media/video-source-load.html	2011-10-07 02:33:23 UTC (rev 96891)
@@ -29,7 +29,7 @@
                 var source = document.createElement('source');
                 source.src = "" name);
                 sources.push(source.src);
-                source.type = mimeTypeForExtension(source.src.split('.')[1]);
+                source.type = mimeTypeForExtension(source.src.split('.').pop());
                 video.appendChild(source);
             }
 

Modified: trunk/LayoutTests/media/video-source-moved.html (96890 => 96891)


--- trunk/LayoutTests/media/video-source-moved.html	2011-10-07 02:23:03 UTC (rev 96890)
+++ trunk/LayoutTests/media/video-source-moved.html	2011-10-07 02:33:23 UTC (rev 96891)
@@ -153,7 +153,7 @@
             {
                 var source = document.createElement('source');
                 source.src = "" index + "-" + Date.now());
-                source.type = mimeTypeForExtension(source.src.split('.')[1]);
+                source.type = mimeTypeForExtension(source.src.split('.').pop());
                 video.appendChild(source);
             }
             

Modified: trunk/LayoutTests/media/video-source-removed.html (96890 => 96891)


--- trunk/LayoutTests/media/video-source-removed.html	2011-10-07 02:23:03 UTC (rev 96890)
+++ trunk/LayoutTests/media/video-source-removed.html	2011-10-07 02:33:23 UTC (rev 96891)
@@ -48,7 +48,7 @@
             {
                 source = document.createElement('source');
                 source.src = "" index + "-" + Date.now());
-                source.type = mimeTypeForExtension(source.src.split('.')[1]);
+                source.type = mimeTypeForExtension(source.src.split('.').pop());
                 video.appendChild(source);
             }
             
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to