Title: [130670] trunk/LayoutTests
Revision
130670
Author
[email protected]
Date
2012-10-08 12:07:33 -0700 (Mon, 08 Oct 2012)

Log Message

Fix video-test.js relativeURL() bug
https://bugs.webkit.org/show_bug.cgi?id=98546

Patch by Shadi Khalek <[email protected]> on 2012-10-08
Reviewed by Eric Seidel.

Some layout tests failed due to this bug (crbug.com/154157)

* media/video-test.js:
(testArraysEqual):
(relativeURL):

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (130669 => 130670)


--- trunk/LayoutTests/ChangeLog	2012-10-08 19:02:06 UTC (rev 130669)
+++ trunk/LayoutTests/ChangeLog	2012-10-08 19:07:33 UTC (rev 130670)
@@ -1,3 +1,16 @@
+2012-10-08  Shadi Khalek  <[email protected]>
+
+        Fix video-test.js relativeURL() bug
+        https://bugs.webkit.org/show_bug.cgi?id=98546
+
+        Reviewed by Eric Seidel.
+
+        Some layout tests failed due to this bug (crbug.com/154157)
+
+        * media/video-test.js:
+        (testArraysEqual):
+        (relativeURL):
+
 2012-10-08  Julien Chaffraix  <[email protected]>
 
         Additional fast/repaint Chromium rebaseline.

Modified: trunk/LayoutTests/media/video-test.js (130669 => 130670)


--- trunk/LayoutTests/media/video-test.js	2012-10-08 19:02:06 UTC (rev 130669)
+++ trunk/LayoutTests/media/video-test.js	2012-10-08 19:07:33 UTC (rev 130670)
@@ -97,7 +97,7 @@
         consoleWrite(ex);
         return;
     }
-  
+
     testExpected(testFuncString + ".length", expected.length);
 
     for (var i = 0; i < observed.length; i++) {
@@ -258,7 +258,7 @@
 
 function relativeURL(url)
 {
-    return url.substr(url.indexOf('/media/')+7);
+    return url.substr(url.lastIndexOf('/media/')+7);
 }
 
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to