Changeset 176990 in webkit

Timestamp:

Dec 8, 2014, 4:16:38 PM (10 years ago)

Author:

ap@apple.com

Message:

REGRESSION (173394): Support for webcam is broken
​https://bugs.webkit.org/show_bug.cgi?id=139313

Test by Antti Koivisto, reviewed and tweaked by Alexey Proskuryakov.

• http/tests/multipart/multipart-image-expected.html: Added.
• http/tests/multipart/multipart-image.html: Added.

• http/tests/multipart/resources/multipart.php: Multiopart boundary must start on

a new line, so make it so. In newwer OS versions, CFNetwork has a workaround that
makes invalid multipart response work, which is why the test was passing locally.

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• http/tests/multipart/multipart-image-expected.html (added)
• http/tests/multipart/multipart-image.html (added)
• http/tests/multipart/resources/multipart.php (modified) (1 diff)

trunk/LayoutTests/ChangeLog ¶

@@ +1 @@
+2014-12-08  Alexey Proskuryakov  <ap@apple.com>
+
+        REGRESSION (173394): Support for webcam is broken
+        https://bugs.webkit.org/show_bug.cgi?id=139313
+
+        Test by Antti Koivisto, reviewed and tweaked by Alexey Proskuryakov.
+
+        * http/tests/multipart/multipart-image-expected.html: Added.
+        * http/tests/multipart/multipart-image.html: Added.
+
+        * http/tests/multipart/resources/multipart.php: Multiopart boundary must start on
+        a new line, so make it so. In newwer OS versions, CFNetwork has a workaround that
+        makes invalid multipart response work, which is why the test was passing locally.
+
 2014-12-08  Benjamin Poulain  <bpoulain@apple.com>
 

trunk/LayoutTests/http/tests/multipart/resources/multipart.php ¶

@@ -8 +8 @@
         echo("Content-Type: image/png\r\n\r\n");
         echo($data);
-        echo("--$boundary\r\n");
+        echo("\r\n--$boundary\r\n");
         flush();
     }