1053927 - Add memory leak check to GrallocTextureHostOGL

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Comment 1

[Sotaro Ikeda [:sotaro OOO until July/21]]

It seems nice to add memory leak check like MemoryTextureHost::~MemoryTextureHost().

http://mxr.mozilla.org/mozilla-central/source/gfx/layers/composite/TextureHost.cpp#645

Comment 2

[Sotaro Ikeda [:sotaro OOO until July/21]]

Attachment: patch - Add memory leak check

Comment 3

[Sotaro Ikeda [:sotaro OOO until July/21]]

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f8df9e1fb07f

Comment 4

[Sotaro Ikeda [:sotaro OOO until July/21]]

(In reply to Sotaro Ikeda [:sotaro] from comment #3)
> https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f8df9e1fb07f

There seems cases that TextureParent::ActorDestroy()s are not called on inter thread IPC case.

Comment 5

[Sotaro Ikeda [:sotaro OOO until July/21]]

Attachment: patch - Add memory leak check

Relax the leaking check.

Comment 6

[Sotaro Ikeda [:sotaro OOO until July/21]]

https://tbpl.mozilla.org/?tree=Try&rev=6cf2bf194f14

Comment 7

[Sotaro Ikeda [:sotaro OOO until July/21]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fe85fa066bf2

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fe85fa066bf2