RESOLVED FIXED 144057
http/tests/websocket/tests/hybi/stop-on-resume-in-error-handler.html crashes on Windows almost all the time
https://bugs.webkit.org/show_bug.cgi?id=144057
Summary http/tests/websocket/tests/hybi/stop-on-resume-in-error-handler.html crashes ...
Alexey Proskuryakov
Reported 2015-04-22 11:33:25 PDT
Attachments
Patch (2.16 KB, patch)
2016-05-30 02:18 PDT, Per Arne Vollan
no flags
Archive of layout-test-results from ews103 for mac-yosemite (872.42 KB, application/zip)
2016-05-30 02:54 PDT, Build Bot
no flags
Patch (1.95 KB, patch)
2016-05-30 07:12 PDT, Per Arne Vollan
bfulgham: review+
Chris Dumez
Comment 1 2015-04-23 10:13:13 PDT
Sadly the Windows bot doesn't give us a trace. I am going to skip the test for now and look into this when I find time.
Chris Dumez
Comment 2 2015-04-23 10:18:12 PDT
Temporarily skipped in <http://trac.webkit.org/changeset/183191>.
Radar WebKit Bug Importer
Comment 3 2015-05-05 17:13:30 PDT
Brent Fulgham
Comment 4 2016-03-22 10:21:08 PDT
This seems to only happen in Release builds.
Per Arne Vollan
Comment 5 2016-05-30 02:18:54 PDT
Build Bot
Comment 6 2016-05-30 02:54:47 PDT
Comment on attachment 280086 [details] Patch Attachment 280086 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/1406444 Number of test failures exceeded the failure limit.
Build Bot
Comment 7 2016-05-30 02:54:52 PDT
Created attachment 280087 [details] Archive of layout-test-results from ews103 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-yosemite Platform: Mac OS X 10.10.5
Per Arne Vollan
Comment 8 2016-05-30 07:12:48 PDT
Brent Fulgham
Comment 9 2016-05-30 09:37:55 PDT
Comment on attachment 280098 [details] Patch Thank you for fixing this! Can you unskip the test when you land the fix?
Per Arne Vollan
Comment 10 2016-05-30 09:40:40 PDT
(In reply to comment #9) > Comment on attachment 280098 [details] > Patch > > Thank you for fixing this! Can you unskip the test when you land the fix? Will do :) Thanks for reviewing!
Per Arne Vollan
Comment 11 2016-05-30 12:05:10 PDT
Per Arne Vollan
Comment 12 2016-05-30 12:06:37 PDT
Updated test expectations in <http://trac.webkit.org/changeset/201501>.
Chris Dumez
Comment 13 2016-05-30 12:12:17 PDT
Comment on attachment 280098 [details] Patch Not sure how the protector helps since it is not capture by value in the lambda?
Per Arne Vollan
Comment 14 2016-05-30 13:03:24 PDT
(In reply to comment #13) > Comment on attachment 280098 [details] > Patch > > Not sure how the protector helps since it is not capture by value in the > lambda? I added this to make sure the SocketStreamHandle object is not deleted before the call is executed on the main thread, and used a RefPtr since the thread will wait for the call to finish. Perhaps this is not needed?
Chris Dumez
Comment 15 2016-05-30 13:39:50 PDT
Oh, I thought this was a callOnMainThread() call. I am not familiar with callOnMainThreadAndWait() but your change is probably fine then.
Per Arne Vollan
Comment 16 2016-05-30 13:56:01 PDT
(In reply to comment #15) > Oh, I thought this was a callOnMainThread() call. I am not familiar with > callOnMainThreadAndWait() but your change is probably fine then. Sounds good, I actually think this file is the only place where callOnMainThreadAndWait() is used :)
Note You need to log in before you can comment on or make changes to this bug.