From 1dca4eb84166d175ca6e43872089efae9a29705e Mon Sep 17 00:00:00 2001 From: Prabir Pradhan Date: Sat, 4 May 2024 00:43:43 +0000 Subject: NativeInputManager: Cleanup after PointerChoreographer refactor Bug: 311416205 Test: build, presubmit Change-Id: Ic52e122f6f2c14f401cea534afef3a7a1822aa40 --- libs/input/PointerController.cpp | 14 -------------- 1 file changed, 14 deletions(-) (limited to 'libs/input/PointerController.cpp') diff --git a/libs/input/PointerController.cpp b/libs/input/PointerController.cpp index 933a33eb8b0e..43264dbe9236 100644 --- a/libs/input/PointerController.cpp +++ b/libs/input/PointerController.cpp @@ -142,7 +142,6 @@ PointerController::PointerController(const sp& PointerController::~PointerController() { mDisplayInfoListener->onPointerControllerDestroyed(); mUnregisterWindowInfosListener(mDisplayInfoListener); - mContext.getPolicy()->onPointerDisplayIdChanged(ADISPLAY_ID_NONE, FloatPoint{0, 0}); } std::mutex& PointerController::getLock() const { @@ -313,12 +312,6 @@ void PointerController::reloadPointerResources() { } void PointerController::setDisplayViewport(const DisplayViewport& viewport) { - struct PointerDisplayChangeArgs { - int32_t displayId; - FloatPoint cursorPosition; - }; - std::optional pointerDisplayChanged; - { // acquire lock std::scoped_lock lock(getLock()); @@ -330,15 +323,8 @@ void PointerController::setDisplayViewport(const DisplayViewport& viewport) { mCursorController.setDisplayViewport(viewport, getAdditionalMouseResources); if (viewport.displayId != mLocked.pointerDisplayId) { mLocked.pointerDisplayId = viewport.displayId; - pointerDisplayChanged = {viewport.displayId, mCursorController.getPosition()}; } } // release lock - - if (pointerDisplayChanged) { - // Notify the policy without holding the pointer controller lock. - mContext.getPolicy()->onPointerDisplayIdChanged(pointerDisplayChanged->displayId, - pointerDisplayChanged->cursorPosition); - } } void PointerController::updatePointerIcon(PointerIconStyle iconId) { -- cgit v1.2.3-59-g8ed1b