commit 95a4ed6e84e9b8845359f601050218fa28459f4b
author Siarhei Vishniakou <svv@google.com> Fri Nov 01 05:53:44 2019 +0000
committer Siarhei Vishniakou <svv@google.com> Fri Nov 01 05:53:44 2019 +0000
tree bff866cfcf0c6f095e151ca4b98b47fb4fd20255
parent ba266f24e580de2d8339050b62fad7e66441f843

Revert "Let InputReader handle its own thread"

This reverts commit ba266f24e580de2d8339050b62fad7e66441f843.

Reason for revert: b/143735040 looks like this CL breaks input tests

Change-Id: I8c19046935d2bdaf9df5df97b2eff43308065c72

services/inputflinger/InputManager.cpp

diff --git a/services/inputflinger/InputManager.cpp b/services/inputflinger/InputManager.cpp
index 34a04df..7d30672 100644
--- a/services/inputflinger/InputManager.cpp
+++ b/services/inputflinger/InputManager.cpp
@@ -46,6 +46,7 @@
 }
 
 void InputManager::initialize() {
+    mReaderThread = new InputReaderThread(mReader);
     mDispatcherThread = new InputDispatcherThread(mDispatcher);
 }
 
@@ -56,9 +57,9 @@
         return result;
     }
 
-    result = mReader->start();
+    result = mReaderThread->run("InputReader", PRIORITY_URGENT_DISPLAY);
     if (result) {
-        ALOGE("Could not start InputReader due to error %d.", result);
+        ALOGE("Could not start InputReader thread due to error %d.", result);
 
         mDispatcherThread->requestExit();
         return result;
@@ -68,9 +69,9 @@
 }
 
 status_t InputManager::stop() {
-    status_t result = mReader->stop();
+    status_t result = mReaderThread->requestExitAndWait();
     if (result) {
-        ALOGW("Could not stop InputReader due to error %d.", result);
+        ALOGW("Could not stop InputReader thread due to error %d.", result);
     }
 
     result = mDispatcherThread->requestExitAndWait();