Closed Bug 1605248 Opened 4 years ago Closed 4 years ago

LoaderObserver::Clear should make the LoaderObserver stop accumulating further data

Categories

(Core :: mozglue, defect)

Unspecified
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: bugzilla, Assigned: bugzilla)

Details

Attachments

(1 file)

Once LoaderObserver::Clear is called, we shouldn't save any more module loads; that is a clear indicator from the client that it has no intention of processing them.

We rename LoaderObserver::Clear to LoaderObserver::Disable to more accurately
reflect the following behaviour change:

Not only does the Disable call free any enqueued module load events, it also
ensures that no further module loads will be saved in the future. This reflects
the reality that any mozglue client that calls Disable has no intention of
ever processing these events.

Pushed by aklotz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7d0f2a59e84e
Convert LoaderObserver::Clear to LoaderObserver::Disable; r=mhowell
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: