Bug 127979

Summary: [Cocoa] Add NEFilterSource support to ContentFilterMac
Product: WebKit Reporter: Andy Estes <aestes>
Component: New BugsAssignee: Andy Estes <aestes>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, sam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
[Cocoa] Add NEFilterSource support to ContentFilterMac
none
[Cocoa] Add NEFilterSource support to ContentFilterMac
none
[Cocoa] Add NEFilterSource support to ContentFilterMac
none
Buffer incoming data in ContentFilter when using NEFilterSource
none
Buffer incoming data in ContentFilter when using NEFilterSource sam: review+

Description Andy Estes 2014-01-30 18:28:06 PST
[Cocoa] Add NEFilterSource support to ContentFilterMac
Comment 1 Andy Estes 2014-01-30 18:28:31 PST
<rdar://problem/15061531>
Comment 2 Andy Estes 2014-01-30 18:49:59 PST
Created attachment 222781 [details]
[Cocoa] Add NEFilterSource support to ContentFilterMac
Comment 3 WebKit Commit Bot 2014-01-30 18:51:54 PST
Attachment 222781 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:62:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:63:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:64:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:65:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:69:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Source/WebCore/platform/mac/ContentFilterMac.mm:70:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 6 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Andy Estes 2014-01-31 13:25:44 PST
Created attachment 222842 [details]
[Cocoa] Add NEFilterSource support to ContentFilterMac
Comment 5 Andy Estes 2014-01-31 13:27:04 PST
Created attachment 222843 [details]
[Cocoa] Add NEFilterSource support to ContentFilterMac
Comment 6 Andy Estes 2014-01-31 17:36:40 PST
Committed r163226: <http://trac.webkit.org/changeset/163226>
Comment 7 Andy Estes 2014-02-04 18:02:51 PST
Created attachment 223192 [details]
Buffer incoming data in ContentFilter when using NEFilterSource
Comment 8 Andy Estes 2014-02-04 18:03:18 PST
Re-opening to post an additional patch.
Comment 9 Andy Estes 2014-02-05 00:42:01 PST
Created attachment 223217 [details]
Buffer incoming data in ContentFilter when using NEFilterSource
Comment 10 Andy Estes 2014-02-05 16:16:06 PST
Committed r163486: <http://trac.webkit.org/changeset/163486>