commit bf88052327da62fceeb07d98ac62f3c8f56047aa
author Siarhei Vishniakou <svv@google.com> Mon May 01 11:03:22 2023 -0700
committer Siarhei Vishniakou <svv@google.com> Mon May 01 11:12:48 2023 -0700
tree 940b93465e77308fe7e1501557b0fbec0171011e
parent ee324d5fe81414b3971ae9579878a70afa8f5c7f

Mark slipWallpaperTouch as const

This function is already not modifying the dispatcher state, so let's
mark it as const. This will allow other functions that depend on it to
also be marked as const in the future.

Other small fixes in this CL:
1. Remove code after fatal log. This code will never execute.
2. Use MotionEvent::getActionMasked

Bug: 273376858
Test: presubmit
Change-Id: I9aaba33f7c3b5dd29a45b120c2823cd41f7f8380

services/inputflinger/dispatcher/InputDispatcher.h

diff --git a/services/inputflinger/dispatcher/InputDispatcher.h b/services/inputflinger/dispatcher/InputDispatcher.h
index dd7f7fe..187429d 100644
--- a/services/inputflinger/dispatcher/InputDispatcher.h
+++ b/services/inputflinger/dispatcher/InputDispatcher.h
@@ -707,8 +707,8 @@
     void slipWallpaperTouch(ftl::Flags<InputTarget::Flags> targetFlags,
                             const sp<android::gui::WindowInfoHandle>& oldWindowHandle,
                             const sp<android::gui::WindowInfoHandle>& newWindowHandle,
-                            TouchState& state, int32_t pointerId, std::vector<InputTarget>& targets)
-            REQUIRES(mLock);
+                            TouchState& state, int32_t pointerId,
+                            std::vector<InputTarget>& targets) const REQUIRES(mLock);
     void transferWallpaperTouch(ftl::Flags<InputTarget::Flags> oldTargetFlags,
                                 ftl::Flags<InputTarget::Flags> newTargetFlags,
                                 const sp<android::gui::WindowInfoHandle> fromWindowHandle,