1078327 - FF needs correct check PointerCapture for correct firing pointerover/pointerout events

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Maksim Lebedev]

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0
Build ID: 20140825202822

Steps to reproduce:

If users want set pointer capture on element, which contain another elements, FF should correct detecting cases for firing pointerover/pointerout events.


Actual results:

If pointer crosses boundaries of internal elements, pointerout event was fired at element which have pointercapture.


Expected results:

pointerout event should be fired only in case if pointer crosses external boundaries of capturing content.

Comment 1

[Maksim Lebedev]

Attachment: check_capturing_content_ver1.diff

+ Add correct checking for detect elements with contents

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8500505 [details] [diff] [review]
check_capturing_content_ver1.diff

We need a test for this.

Comment 3

[Maksim Lebedev]

(In reply to Olli Pettay [:smaug] from comment #2)
> We need a test for this.
I will try, but I am very doubt that I can provide auto test.

Comment 4

[Maksim Lebedev]

Attachment: check_capturing_content_test_ver1.diff

+ Test for correct detecting capturing content

Comment 5

[Maksim Lebedev]

Test without patch:
https://tbpl.mozilla.org/?tree=Try&rev=27f83657a1cd
Test with patch:
https://tbpl.mozilla.org/?tree=Try&rev=799537b8932d

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

I see failures on try.

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8501675 [details] [diff] [review]
check_capturing_content_test_ver1.diff

Please fix the failures.

Comment 8

[Maksim Lebedev]

Attachment: check_capturing_content_test_ver2.diff

+ Changes: Add correct finish for test

Comment 9

[Maksim Lebedev]

Test without patch:
https://tbpl.mozilla.org/?tree=Try&rev=585b9e3e2b4e
Test with patch:
https://tbpl.mozilla.org/?tree=Try&rev=8f2c1924d40a

Comment 10

[Maksim Lebedev]

If nobody have objections, I put checkin flag.

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/458b83f2eaf4
https://hg.mozilla.org/integration/mozilla-inbound/rev/70868ad1299f

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/458b83f2eaf4
https://hg.mozilla.org/mozilla-central/rev/70868ad1299f