1700251 - Some light refactoring in APZCTreeManager::ReceiveInputEvent()

Status: RESOLVED FIXED

(Core :: Panning and Zooming, task)

Comment 1

[Botond Ballo [:botond]]

Attachment: Bug 1700251 - Introduce InputHandlingState. r=tnikkel

Comment 2

[Botond Ballo [:botond]]

Attachment: Bug 1700251 - Inline the local hitResult variable in ReceiveInputEvent(). r=tnikkel

Depends on D109457

Comment 3

[Botond Ballo [:botond]]

Attachment: Bug 1700251 - Factor out a HitTestResult::CopyWithoutScrollbarNode() helper. r=tnikkel

Depends on D109458

Comment 4

[Botond Ballo [:botond]]

Attachment: Bug 1700251 - Move the HitTestResult into the InputHandlingState. r=tnikkel

Depends on D109459

Comment 5

[Botond Ballo [:botond]]

Attachment: Bug 1700251 - Move the code to set the LayersId on the event into InputHandlingResult::Finish(). r=tnikkel

This allows that code (and in the future, other code that needs
to run for every input type) to be in just one place.

Depends on D109460

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6b49a6811451
Introduce InputHandlingState. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/78f4dd6c8ad9
Inline the local hitResult variable in ReceiveInputEvent(). r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/bf79b5a976ad
Factor out a HitTestResult::CopyWithoutScrollbarNode() helper. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/bd6114b672bd
Move the HitTestResult into the InputHandlingState. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/6b439669e688
Move the code to set the LayersId on the event into InputHandlingResult::Finish(). r=tnikkel

Comment 7

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/6b49a6811451
https://hg.mozilla.org/mozilla-central/rev/78f4dd6c8ad9
https://hg.mozilla.org/mozilla-central/rev/bf79b5a976ad
https://hg.mozilla.org/mozilla-central/rev/bd6114b672bd
https://hg.mozilla.org/mozilla-central/rev/6b439669e688