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+

Description Dirk Pranke 2011-06-02 17:47:44 PDT
nrwt: fix duplicate logging regression
Comment 1 Dirk Pranke 2011-06-02 17:48:10 PDT
Created attachment 95841 [details]
Patch
Comment 2 Dirk Pranke 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 :)
Comment 3 Dirk Pranke 2011-06-02 17:53:57 PDT
Comment on attachment 95841 [details]
Patch

whoops. this patch won't work.
Comment 4 Dirk Pranke 2011-06-02 17:56:01 PDT
Created attachment 95842 [details]
Patch
Comment 5 Dirk Pranke 2011-06-03 12:00:18 PDT
Committed r88035: <http://trac.webkit.org/changeset/88035>