Changeset 130670 in webkit

Timestamp:

Oct 8, 2012, 12:07:33 PM (14 years ago)

Author:

commit-queue@webkit.org

Message:

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

Patch by Shadi Khalek <​shadi@chromium.org> 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):

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• media/video-test.js (modified) (2 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2012-10-08  Shadi Khalek  <shadi@chromium.org>
+
+        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  <jchaffraix@webkit.org>
 

trunk/LayoutTests/media/video-test.js

@@ -98 +98 @@
         return;
     }
-  
+
     testExpected(testFuncString + ".length", expected.length);
 
@@ -259 +259 @@
 function relativeURL(url)
 {
-    return url.substr(url.indexOf('/media/')+7);
+    return url.substr(url.lastIndexOf('/media/')+7);
 }