Summary: | Add basic pattern matching support to the url filters | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Benjamin Poulain <benjamin> | ||||||
Component: | New Bugs | Assignee: | Benjamin Poulain <benjamin> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | barraclough, kling | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Benjamin Poulain
2015-01-08 20:42:31 PST
Created attachment 244322 [details]
Patch
Comment on attachment 244322 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244322&action=review > Source/WebCore/contentextensions/URLFilterParser.cpp:140 > + void disjunction() { } Shouldn't these fail()? (In reply to comment #2) > Comment on attachment 244322 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=244322&action=review > > > Source/WebCore/contentextensions/URLFilterParser.cpp:140 > > + void disjunction() { } > > Shouldn't these fail()? I can add that. I was just implementing the minimum I need at the moment. Eventually everything except backreference should have an implementation. Created attachment 244370 [details]
Patch
Comment on attachment 244370 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244370&action=review r=me > Source/WebCore/contentextensions/URLFilterParser.cpp:70 > + ASSERT(isASCII(character)); I think this needs a runtime check, there's no guarantee that 'character' will be ASCII. Committed r178313: <http://trac.webkit.org/changeset/178313> |