1792574 - deCOM nsIMemory

Status: RESOLVED FIXED

(Core :: XPCOM, task, P3)

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1792574, part 1 - Move nsAssignmentType to nsStringStream.h.

That's the only place it is used.

Comment 2

[Andrew McCreight [:mccr8]]

Attachment: Bug 1792574, part 2 - Remove some unused macros from nsMemory.h.

Comment 3

[Andrew McCreight [:mccr8]]

Attachment: Bug 1792574, part 3 - Don't include nsMemory where it isn't needed.

There are only 3 places where nsMemory.h is still needed (image/RasterImage.cpp,
gfx/thebes/gfxFT2FontList.cpp, and nsMemory.cpp). Remove the rest.

Comment 4

[Andrew McCreight [:mccr8]]

Attachment: Bug 1792574, part 4 - deCOM nsIMemory.

I left the implementation in nsMemoryImpl.cpp so that the
diff looks reasonable.

I also got rid of the weird singleton runnable and replaced
it with NS_NewRunnableFunction.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7cd23855285a
part 1 - Move nsAssignmentType to nsStringStream.h. r=xpcom-reviewers,nika
https://hg.mozilla.org/integration/autoland/rev/ced246802273
part 2 - Remove some unused macros from nsMemory.h. r=xpcom-reviewers,nika
https://hg.mozilla.org/integration/autoland/rev/5c78a73fd7cd
part 3 - Don't include nsMemory where it isn't needed. r=xpcom-reviewers,necko-reviewers,valentin,nika
https://hg.mozilla.org/integration/autoland/rev/1f5cc8351a05
part 4 - deCOM nsIMemory. r=xpcom-reviewers,nika

Comment 6

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/7cd23855285a
https://hg.mozilla.org/mozilla-central/rev/ced246802273
https://hg.mozilla.org/mozilla-central/rev/5c78a73fd7cd
https://hg.mozilla.org/mozilla-central/rev/1f5cc8351a05