RESOLVED FIXED Bug 67530
Reshuffle some code in WebKitDriver._read_block in preparation for reading stderr/stdout separately
https://bugs.webkit.org/show_bug.cgi?id=67530
Summary Reshuffle some code in WebKitDriver._read_block in preparation for reading st...
Eric Seidel (no email)
Reported 2011-09-02 15:13:03 PDT
Reshuffle some code in WebKitDriver._read_block in preparation for reading stderr/stdout separately
Attachments
Patch (4.15 KB, patch)
2011-09-02 15:15 PDT, Eric Seidel (no email)
abarth: review+
webkit.review.bot: commit-queue-
Eric Seidel (no email)
Comment 1 2011-09-02 15:15:24 PDT
Eric Seidel (no email)
Comment 2 2011-09-02 15:19:23 PDT
Ryosuke Niwa
Comment 4 2011-09-03 00:02:11 PDT
I have confirmed that this patch is causing the failures. Since these failures are preventing the rest of tests to run, I'm going to rollout the patch for now unless either one of you can fix it in the next 10 minutes or so.
Ryosuke Niwa
Comment 5 2011-09-03 00:29:04 PDT
WebKit Review Bot
Comment 6 2011-09-03 00:32:13 PDT
Comment on attachment 106205 [details] Patch Rejecting attachment 106205 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: Scripts/svn-apply', u'--reviewer', u'Adam Barth', u'--force']" exit_code: 1 Parsed 2 diffs from patch file(s). patching file Tools/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file Tools/Scripts/webkitpy/layout_tests/port/webkit.py Hunk #1 FAILED at 509. 1 out of 1 hunk FAILED -- saving rejects to file Tools/Scripts/webkitpy/layout_tests/port/webkit.py.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Adam Barth', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/9587229
Ryosuke Niwa
Comment 7 2011-09-03 00:44:49 PDT
Also rolled out the unreviewed followup in http://trac.webkit.org/changeset/94483.
Eric Seidel (no email)
Comment 8 2011-09-03 01:17:41 PDT
Thanks Ryosuke.
Eric Seidel (no email)
Comment 9 2011-09-09 11:43:11 PDT
I guess I'll try this again. :)
Eric Seidel (no email)
Comment 10 2011-09-12 12:59:51 PDT
I found the bug in the original patch, fixed. I also realize now that this is subtly changing behavior. Before we were not correctly respecting the deadline. This should make our timeouts much more accurate, as we're re-computing when the timeout is every time we read a line, instead of computing up front that we should wait 30 seconds, and then making every read wait 30 seconds. The old code would wait forever for a test which slowly produced output.
Eric Seidel (no email)
Comment 11 2011-09-12 13:05:45 PDT
Note You need to log in before you can comment on or make changes to this bug.