Title: [148890] trunk/Source/WebCore
Revision
148890
Author
[email protected]
Date
2013-04-22 09:43:57 -0700 (Mon, 22 Apr 2013)

Log 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 <[email protected]> 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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (148889 => 148890)


--- trunk/Source/WebCore/ChangeLog	2013-04-22 16:40:24 UTC (rev 148889)
+++ trunk/Source/WebCore/ChangeLog	2013-04-22 16:43:57 UTC (rev 148890)
@@ -1,3 +1,17 @@
+2013-04-22  John Griggs  <[email protected]>
+
+        [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  <[email protected]>
 
 

Modified: trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp (148889 => 148890)


--- trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp	2013-04-22 16:40:24 UTC (rev 148889)
+++ trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp	2013-04-22 16:43:57 UTC (rev 148890)
@@ -183,8 +183,8 @@
         mediaURL.setProtocol("file");
         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);
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to