1686037 - leak on layout/base/tests/test_bug993936.html with implicit pointer capture enabled

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[Edgar Chen [:edgar]]

Bug 1682527 was backed out due to there is a leak in https://searchfox.org/mozilla-central/source/layout/base/tests/test_bug993936.html.

Comment 1

[Edgar Chen [:edgar]]

The test synthesizes touchstart and touchmove event, but no touchend. And it seems the PointerCapture isn't released properly due to the leak of touchend event.

Comment 2

[Edgar Chen [:edgar]]

Attachment: Bug 1686037 - Part 1: Move nsGenericHTMLElement::GetPresContext to Element;

Comment 3

[Edgar Chen [:edgar]]

Attachment: Bug 1686037 - Part 2: Clean up pointer capture info when PresContext is destroyed;

Depends on D102321

Comment 4

[Edgar Chen [:edgar]]

Attachment: Bug 1686037 - Part 3: Remove duplicated ImplicitlyReleasePointerCapture call in PresShell;

Depends on D102403

Comment 5

[Edgar Chen [:edgar]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d300f9ca249a6bdcae935ed38664f3f4c6972d3

Comment 6

[Edgar Chen [:edgar]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d01d6dca2c13cfb79c9a6969cce5ea27f904219

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7e5ee5c59695
Part 1: Move nsGenericHTMLElement::GetPresContext to Element; r=smaug
https://hg.mozilla.org/integration/autoland/rev/554546208c0b
Part 2: Clean up pointer capture info when PresContext is destroyed; r=smaug
https://hg.mozilla.org/integration/autoland/rev/8af0c61efbc5
Part 3: Remove duplicated ImplicitlyReleasePointerCapture call in PresShell; r=smaug

Comment 8

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/7e5ee5c59695
https://hg.mozilla.org/mozilla-central/rev/554546208c0b
https://hg.mozilla.org/mozilla-central/rev/8af0c61efbc5