Bug 1600661 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Revision 1 by
on 
Right now we have a lot of duplicated code between `findAllMessages`and `findMessage` as well as `findAllMatchingMessages` and `findMatchingMessage` in ASRouterTargeting.jsm

Let's refactor them to only use a single path i.e. go through `handleMessageRequest` and `findMatchingMessage`
Revision 2 by
on 
Right now we have a lot of duplicated code between `findAllMessages`and `findMessage` as well as `findAllMatchingMessages` and `findMatchingMessage` in ASRouterTargeting.jsm

Let's refactor them to only use a single path i.e. go through `handleMessageRequest` and `findMatchingMessage`. This will help us consolidate some sorting logic into one place eventually.

Back to Bug 1600661 Comment 0