| commit | 19274bdda7b8038580d27020098b5a0c3941d985 | |
| author | mukai <mukai@chromium.org> | |
| Tue, 24 Feb 2015 22:37:48 +0000 (24 14:37 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 24 Feb 2015 22:39:05 +0000 (24 22:39 +0000) | ||
| tree | 80abe8acdcf790e9f9129e0ed7a6c7d40c330a2b | treesnapshot (tar.gz zip) |
| parent | ef7aa83795e50c03bf05bbdaad3fdebbef8036b3 | commitdiff |
Update the event filter order about partial screenshot.
PartialScreenshotController prepends itself to the event filters.
This means that mouse events are handled prior to any other
event filters including mouse cursor event filter which controls
the bigger cursor or the cursor in the software mirroring display.
So we need to be careful of the event filter order, mouse cursor
event filter should be processed first, and then partial screenshot
may handle the event. This means partial screenshot event handler
has to be created in Shell and added to event filters prior to
mouse cursor event filter (because they are prepended).
BUG=459214
R=oshima@chromium.org
TEST=The new test case covers
Review URL: https://codereview.chromium.org/942083006
Cr-Commit-Position: refs/heads/master@{#317904}
PartialScreenshotController prepends itself to the event filters.
This means that mouse events are handled prior to any other
event filters including mouse cursor event filter which controls
the bigger cursor or the cursor in the software mirroring display.
So we need to be careful of the event filter order, mouse cursor
event filter should be processed first, and then partial screenshot
may handle the event. This means partial screenshot event handler
has to be created in Shell and added to event filters prior to
mouse cursor event filter (because they are prepended).
BUG=459214
R=oshima@chromium.org
TEST=The new test case covers
Review URL: https://codereview.chromium.org/942083006
Cr-Commit-Position: refs/heads/master@{#317904}