Bug 61981

Summary: nrwt: fix duplicate logging regression
Product: WebKit Reporter: Dirk Pranke <dpranke>
Component: New BugsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, tony
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch tony: review+

Dirk Pranke
Reported 2011-06-02 17:47:44 PDT
nrwt: fix duplicate logging regression
Attachments
Patch (1.85 KB, patch)
2011-06-02 17:48 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
Patch (1.98 KB, patch)
2011-06-02 17:56 PDT, Dirk Pranke 		tony: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2011-06-02 17:48:10 PDT
Created attachment 95841 [details] Patch
Dirk Pranke
Comment 2 2011-06-02 17:49:45 PDT
I apparently re-introduced the "duplicate logging" issue due to a bad merge in bug 60328. This fixes it. There's no test for this because it's difficult to do given the need for multiple processes and actual shared pipes. Good thing it's trivially easy to verify by eye :)
Dirk Pranke
Comment 3 2011-06-02 17:53:57 PDT
Comment on attachment 95841 [details] Patch whoops. this patch won't work.
Dirk Pranke
Comment 4 2011-06-02 17:56:01 PDT
Created attachment 95842 [details] Patch
Dirk Pranke
Comment 5 2011-06-03 12:00:18 PDT
Committed r88035: <http://trac.webkit.org/changeset/88035>
Note You need to log in before you can comment on or make changes to this bug.