Convert CC analyzer to a .jsm

RESOLVED WONTFIX

Status

Testing
Mochitest
RESOLVED WONTFIX
3 years ago
2 years ago

People

(Reporter: mccr8, Assigned: mccr8)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(7 attachments)

(Assignee)

Description

3 years ago
In order to call the CC analyzer in a child process script, we need to convert it to something that can be run in a non-window context.
(Assignee)

Updated

3 years ago
Assignee: nobody → continuation
(Assignee)

Comment 1

3 years ago
Created attachment 8587739 [details] [diff] [review]
part 1 - Rename url to id, it is tracking window ids.

WIP
(Assignee)

Comment 2

3 years ago
Created attachment 8587740 [details] [diff] [review]
part 2 - Filter out non-opened windows later to eliminate dependence of checkForLeakedGlobalWindows on |this|.
(Assignee)

Comment 3

3 years ago
Created attachment 8587741 [details] [diff] [review]
part 3 - Turn checkForLeakedGlobalWindows into a method on CC analyzer.
(Assignee)

Comment 4

3 years ago
Created attachment 8587742 [details] [diff] [review]
part 4 - Move finish into reportLeaks to eliminate dependence on |this|.
(Assignee)

Comment 5

3 years ago
Created attachment 8587743 [details] [diff] [review]
part 5 - Turn the outer checkForLeakedGlobalWindows into a method in cc analyzer.
(Assignee)

Comment 6

3 years ago
Created attachment 8587744 [details] [diff] [review]
part 6 - Remove dependence on window utils.
(Assignee)

Comment 7

3 years ago
Created attachment 8587745 [details] [diff] [review]
part 7 - Do the conversion to a jsm.
(Assignee)

Comment 8

2 years ago
I'm just going to remove the CCAnalyzer.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.