Closed Bug 1381125 Opened 2 years ago Closed 2 years ago

Add whether a focus target has key listeners to the APZ keyboard logs.

Categories

(Core :: Panning and Zooming, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Attached patch apz-log.patchSplinter Review
This would be useful to have. The bug number on the patch is wrong, I'll fix it before pushing.
Attachment #8886684 - Flags: review?(bugmail)
Comment on attachment 8886684 [details] [diff] [review]
apz-log.patch

Review of attachment 8886684 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/layers/apz/src/FocusTarget.cpp
@@ +121,2 @@
>               aFocusSequenceNumber,
> +             static_cast<int>(mFocusHasKeyEventListeners),

I think the cast is implied, so you can get rid of it you want. Doesn't matter to me either way.
Attachment #8886684 - Flags: review?(bugmail) → review+
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ae7497c1b55
Add whether a focus target has key listeners to the APZ keyboard logs. r=kats
https://hg.mozilla.org/mozilla-central/rev/6ae7497c1b55
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.