Changeset 148890 in webkit

Timestamp:

Apr 22, 2013 9:43:57 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[BlackBerry] URL decode data: URLs before passing to the platform media player.
​https://bugs.webkit.org/show_bug.cgi?id=114887

Patch by John Griggs <​jgriggs@blackberry.com> on 2013-04-22
Reviewed by Rob Buis.

Pass data URL through URL decoding before passing to platform media player. PR 313635.

Reviewed internally by Liam Quinn

• platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp:

(WebCore::MediaPlayerPrivate::load):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-04-22  John Griggs  <jgriggs@blackberry.com>
+
+        [BlackBerry] URL decode data: URLs before passing to the platform media player.
+        https://bugs.webkit.org/show_bug.cgi?id=114887
+
+        Reviewed by Rob Buis.
+
+        Pass data URL through URL decoding before passing to platform media player.  PR 313635.
+
+        Reviewed internally by Liam Quinn
+
+        * platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp:
+        (WebCore::MediaPlayerPrivate::load):
+
 2013-04-22  Xuefei Ren  <xren@blackberry.com>
 

trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp

@@ -184 +184 @@
         mediaURL.setPath(fsRoot + "/" + secOrigin + "/" + fsPath);
         modifiedUrl = mediaURL.string();
-    } else if (modifiedUrl.startsWith("file://")) {
-        // The QNX Multimedia Framework cannot handle filenames containing URL escape sequences.
+    } else if (modifiedUrl.startsWith("file://") || modifiedUrl.startsWith("data:")) {
+        // The QNX Multimedia Framework cannot handle filenames or data containing URL escape sequences.
         modifiedUrl = decodeURLEscapeSequences(modifiedUrl);
     }