Description
Antti Koivisto
2014-12-05 13:56:28 PST
Created attachment 242662 [details]
patch
Comment on attachment 242662 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=242662&action=review > Source/WebKit2/NetworkProcess/NetworkResourceLoader.cpp:200 > + // For multipart/x-mixed-replace didReceiveResponseAsync gets called multiple times and buffering would require special handling. > + if (response.isMultipart() && !isSynchronous()) > + m_bufferedData = nullptr; I'm not sure if I understand what happens (or what should happen) for sync loads. Anyway, this would probably read a little better if the new code was inside an else clause below - there is no need to check isSynchronous twice. > LayoutTests/ChangeLog:13 > + The last image is repeated due to a WebCore side bug that causes last part to not render. This may be appropriate to have in an HTML comment inside the test, as that's where one would be looking when they notice this strangeness. > LayoutTests/http/tests/multipart/multipart-image.html:10 > + if (window.testRunner) > + setTimeout("testRunner.notifyDone()", 1000); Can we watch image size changes to finish faster and more reliably? One second is a long time to waste, but may be not enough to avoid random failures. (In reply to comment #3) > I'm not sure if I understand what happens (or what should happen) for sync > loads. I think it will just smash all data together and deliver it if the stream ever ends. Don't know what should happen. Implemented the other changes. Created attachment 242668 [details]
updated patch
Comment on attachment 242668 [details] updated patch View in context: https://bugs.webkit.org/attachment.cgi?id=242668&action=review > LayoutTests/http/tests/multipart/multipart-image.html:24 > +<!-- The last image is repeated due to a WebCore side bug that causes last part to not render. --> https://bugs.webkit.org/show_bug.cgi?id=36536 ? Comment on attachment 242668 [details] updated patch Attachment 242668 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/4603433885630464 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242678 [details]
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-13 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Comment on attachment 242668 [details] updated patch Attachment 242668 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/4614285321830400 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242679 [details]
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Created attachment 242709 [details]
some changes to try to make the test work in the bots
Comment on attachment 242709 [details] some changes to try to make the test work in the bots Attachment 242709 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5112844924223488 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242711 [details]
Archive of layout-test-results from ews100 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Comment on attachment 242709 [details] some changes to try to make the test work in the bots Attachment 242709 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/5265665967521792 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242712 [details]
Archive of layout-test-results from webkit-ews-11 for mac-mountainlion-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-11 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
*** Bug 139378 has been marked as a duplicate of this bug. *** Created attachment 242788 [details]
another attempt with the test
Comment on attachment 242788 [details] another attempt with the test Attachment 242788 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/5039256766513152 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242790 [details]
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-13 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Created attachment 242791 [details]
another
Comment on attachment 242791 [details] another Attachment 242791 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/6380347419787264 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242795 [details]
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Comment on attachment 242791 [details] another Attachment 242791 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/4787785894461440 New failing tests: http/tests/multipart/multipart-image.html Created attachment 242799 [details]
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-03 Port: mac-mountainlion Platform: Mac OS X 10.8.5
https://trac.webkit.org/r176947 Landed without the test since I can't figure out how to make it work in bots (any bots, including those not affected by the patch). It is as if the php script that generated the multipart replies doesn't run correctly. Re-opening for a test. This is actually a pre-existent problem in multipart.php, which produces invalid multipart responses (which newer CFNetwork now supports). Created attachment 242833 [details]
updated test
Let's see what EWS thinks.
Attachment 242833 [details] did not pass style-queue:
ERROR: LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5]
Total errors found: 1 in 4 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Committed the test in <http://trac.webkit.org/r176990>. Thank you Alexey. |