Opened on 08/29/2014 at 01:08:49 PM
Closed on 09/17/2014 at 02:35:34 PM
#1286 closed change (fixed)
Add a content type enum
Reported by: | fhd | Assignee: | hexene |
---|---|---|---|
Priority: | P4 | Milestone: | |
Module: | Libadblockplus | Keywords: | |
Cc: | trev | Blocked By: | |
Blocking: | Platform: | Unknown | |
Ready: | yes | Confidential: | no |
Tester: | Verified working: | no | |
Review URL(s): |
Description
Background
FilterEngine::Matches currently gets contentType as a string. Since there's a fixed list of supported content types, it should rather be an enum.
What to change
Create a ContentType enum and use it in FilterEngine::Matches.
Attachments (0)
Change History (3)
comment:1 Changed on 09/01/2014 at 02:05:09 PM by fhd
- Owner set to hexene
comment:2 Changed on 09/02/2014 at 09:15:30 PM by trev
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 09/17/2014 at 02:35:34 PM by fhd
- Resolution set to fixed
- Status changed from reviewing to closed
Note: See
TracTickets for help on using
tickets.
Pushed: https://hg.adblockplus.org/libadblockplus/rev/e4c8240d0d1f