RESOLVED FIXED 91601
Regression (r122913) webkitpy tests are failing 
https://bugs.webkit.org/show_bug.cgi?id=91601
Summary Regression (r122913) webkitpy tests are failing
Thiago Marcos P. Santos
Reported 2012-07-18 01:12:49 PDT
[0/1447] webkit2.messages_unittest.HeaderTest.test_header [689/1447] webkitpy.layout_tests.port.server_process_unittest.TestServerProcess.test_broken_pipe erred: Traceback (most recent call last): File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/port/server_process_unittest.py", line 129, in test_broken_pipe server_process.write("should break") File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/port/server_process.py", line 144, in write self.stop() File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/port/server_process.py", line 331, in stop timeout = time.time() + self._port.process_kill_time() AttributeError: 'TrivialMockPort' object has no attribute 'process_kill_time'
Attachments
Patch (1.33 KB, patch)
2012-07-18 01:18 PDT, Thiago Marcos P. Santos 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Thiago Marcos P. Santos
Comment 1 2012-07-18 01:18:09 PDT
Created attachment 152959 [details] Patch
Kristóf Kosztyó
Comment 2 2012-07-18 01:23:55 PDT
Sorry I didn't notice that you working on this. I was already done this in r122913 (http://trac.webkit.org/changeset/122932)
Kristóf Kosztyó
Comment 3 2012-07-18 01:26:49 PDT
(In reply to comment #2) > Sorry I didn't notice that you working on this. I was already done this in r122913 (http://trac.webkit.org/changeset/122932) And I have some problems with the numbers :) r122932
Thiago Marcos P. Santos
Comment 4 2012-07-18 01:39:42 PDT
(In reply to comment #2) > Sorry I didn't notice that you working on this. I was already done this in r122913 (http://trac.webkit.org/changeset/122932) No problem. But maybe next time would be nice to follow the standard patch submitting flow to avoid this kind of duplication.
Note You need to log in before you can comment on or make changes to this bug.