1524239 - Attach the LayersId of the process-top-level layer that has focus to keyboard events

Status: RESOLVED FIXED

(Core :: Panning and Zooming, enhancement, P3)

Description

[Henri Sivonen (:hsivonen)]

In APZCTreeManager::ReceiveInputEvent attach the LayersId from bug 1524236 to the keyboard event using the field from bug 1524226.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

So one question is whether we actually need to store this information in the input event itself. Right now the results of the APZ hit test are returned to the caller via the ScrollableLayerGuid out-param in ReceiveInputEvent code and the guid contains the LayersId. We also have this InputAPZContext mechanism that we use in a bunch of places (e.g. here) to stash the APZ hit-test result on the stack and then access it like so. In this example it's used to propagate it from TabParent to TabChild, but we could just as easily use it in the DispatchCrossProcessEvent code to figure out which content process to send it to.

Comment 2

[Botond Ballo [:botond]]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #1)

So one question is whether we actually need to store this information in the input event itself. Right now the results of the APZ hit test are returned to the caller via the ScrollableLayerGuid out-param in ReceiveInputEvent code and the guid contains the LayersId.

That might not be the right LayersId, though.

Consider a mouse event over a non-scrollable OOP iframe inside a scrollable parent frame. The guid computed for APZ purposes will reference the nearest enclosing scrollable layer, which in this case is the parent, but presumably we want to dispatch the event to the process of the OOP iframe.

Comment 3

[Botond Ballo [:botond]]

Oh, this bug is about keyboard events.

I think my point is still valid, but should have been made in bug 1524226 instead :)

Comment 4

[Henri Sivonen (:hsivonen)]

Attachment: Bug 1524239 - Attach the LayersId of the process-top-level layer that has focus to keyboard events.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/67540efd30a3
Attach the LayersId of the process-top-level layer that has focus to keyboard events. r=botond

Comment 6

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/67540efd30a3