Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Windows] Synthesize modifier keys events on pointer events #38138

Conversation

bleroux
Copy link
Contributor

@bleroux bleroux commented Dec 8, 2022

Description

This PR implements modifier keys states synchronization based on pointer events.
It is the Windows implementation for flutter/flutter#115066.
Win32 mouse events only expose flags for Shift and Control, so only those two modifiers are sync.

Related Issue

flutter/flutter#115066

Tests

Adds 4 tests.

@bleroux bleroux force-pushed the windows_synthesize_modifier_keys_event_on_pointer_event branch 2 times, most recently from f818f94 to 2b1eaf1 Compare December 8, 2022 20:40
@bleroux bleroux force-pushed the windows_synthesize_modifier_keys_event_on_pointer_event branch from 2b1eaf1 to 2e84c29 Compare December 9, 2022 09:00
@bleroux bleroux requested a review from dkwingsmt December 9, 2022 09:44
Copy link
Contributor

@dkwingsmt dkwingsmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@bleroux bleroux added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 15, 2022
@auto-submit auto-submit bot merged commit 6a2560c into flutter:main Dec 15, 2022
@bleroux bleroux deleted the windows_synthesize_modifier_keys_event_on_pointer_event branch December 15, 2022 07:26
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 15, 2022
…117148)

* df430c4fd Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter/engine#38292)

* 41dd4f5e1 Revert "Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (#38292)" (flutter/engine#38301)

* 8ce9a3f71 Roll Skia from 3171deabd88a to b368746d696a (13 revisions) (flutter/engine#38294)

* 4881fe25e Revert "Revert "reland "Migrate darwin common "framework_shared" target to ARC #37049" (#37219)" (#37320)" (flutter/engine#37883)

* 3b2302c8d [Impeller] Remove validation log when the pipeline library is collected before pipeline is setup. (flutter/engine#38306)

* a04997c81 [Impeller] Disable impeller_unittests. (flutter/engine#38307)

* fc71faad0 License script improvements (flutter/engine#38148)

* 6a2560c35 [Windows] Synthesize modifier keys events on pointer events (flutter/engine#38138)

* b1d407563 Roll Skia from b368746d696a to 3f81f95176ce (11 revisions) (flutter/engine#38312)

* b25fcf748 Roll Skia from 3f81f95176ce to 46e8f2a18a3d (3 revisions) (flutter/engine#38314)

* 948699bba Collapse bounds calculations into DisplayListBuilder (flutter/engine#34365)

* 38367de84 Roll Fuchsia Mac SDK from u-tC0QEGUT4xQ4KOo... to VEOIaacOA75U7PYyz... (flutter/engine#38316)

* 29196519c Roll Skia from 46e8f2a18a3d to 9f728d78f10d (1 revision) (flutter/engine#38317)
loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Dec 16, 2022
loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Jan 3, 2023
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…lutter#117148)

* df430c4fd Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter/engine#38292)

* 41dd4f5e1 Revert "Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter#38292)" (flutter/engine#38301)

* 8ce9a3f71 Roll Skia from 3171deabd88a to b368746d696a (13 revisions) (flutter/engine#38294)

* 4881fe25e Revert "Revert "reland "Migrate darwin common "framework_shared" target to ARC flutter#37049" (flutter#37219)" (flutter#37320)" (flutter/engine#37883)

* 3b2302c8d [Impeller] Remove validation log when the pipeline library is collected before pipeline is setup. (flutter/engine#38306)

* a04997c81 [Impeller] Disable impeller_unittests. (flutter/engine#38307)

* fc71faad0 License script improvements (flutter/engine#38148)

* 6a2560c35 [Windows] Synthesize modifier keys events on pointer events (flutter/engine#38138)

* b1d407563 Roll Skia from b368746d696a to 3f81f95176ce (11 revisions) (flutter/engine#38312)

* b25fcf748 Roll Skia from 3f81f95176ce to 46e8f2a18a3d (3 revisions) (flutter/engine#38314)

* 948699bba Collapse bounds calculations into DisplayListBuilder (flutter/engine#34365)

* 38367de84 Roll Fuchsia Mac SDK from u-tC0QEGUT4xQ4KOo... to VEOIaacOA75U7PYyz... (flutter/engine#38316)

* 29196519c Roll Skia from 46e8f2a18a3d to 9f728d78f10d (1 revision) (flutter/engine#38317)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants