Opened on 03/13/2014 at 01:12:06 PM
Closed on 09/16/2014 at 09:45:16 PM
Last modified on 10/13/2014 at 11:12:28 AM
#138 closed change (fixed)
[meta] Remove timeline code
Reported by: | trev | Assignee: | |
---|---|---|---|
Priority: | P4 | Milestone: | Adblock-Plus-2.6.5-for-Firefox |
Module: | Adblock-Plus-for-Firefox | Keywords: | meta |
Cc: | Blocked By: | #139, #140, #1375, #1380 | |
Blocking: | Platform: | Firefox | |
Ready: | yes | Confidential: | no |
Tester: | Verified working: | no | |
Review URL(s): |
Description
Background
We currently have time measurement code in our Firefox code base that we patch out in release builds. That code has been written back when most of Adblock Plus code was synchronous and doesn't help much now that Adblock Plus uses asynchronous operations. A proper profiler does a better job here.
What to change
Ticket | Status | Resolution | Summary | Owner |
---|---|---|---|---|
#139 | closed | fixed | [Remove timeline code] Remove timeline.js and callers | beelzy |
#140 | closed | fixed | [Remove timeline code] Remove timeline-specific build hacks | |
#1375 | closed | fixed | Remove timeline-specific conversion code from JSHydra | |
#1380 | closed | fixed | Update jshydra dependency in buildtools repository to revision 90ac35f240c2 | trev |
Attachments (0)
Change History (7)
comment:1 Changed on 03/13/2014 at 01:12:48 PM by trev
- Blocked By 139 added
comment:2 Changed on 03/13/2014 at 01:14:45 PM by trev
- Blocked By 140 added
comment:3 Changed on 07/09/2014 at 12:38:11 PM by philll
- Platform set to Firefox
comment:4 Changed on 09/15/2014 at 11:14:56 AM by trev
- Blocked By 1375 added
comment:5 Changed on 09/16/2014 at 09:45:01 PM by trev
- Blocked By 1380 added
comment:6 Changed on 09/16/2014 at 09:45:16 PM by trev
- Ready unset
- Resolution set to fixed
- Status changed from new to closed
comment:7 Changed on 10/13/2014 at 11:12:28 AM by trev
- Milestone set to Adblock-Plus-for-Firefox-next
- Ready set
Note: See
TracTickets for help on using
tickets.