summaryrefslogtreecommitdiff
path: root/libs/gui/LayerState.cpp
diff options
context:
space:
mode:
author Vishnu Nair <vishnun@google.com> 2021-12-01 17:21:16 -0800
committer Vishnu Nair <vishnun@google.com> 2021-12-03 17:00:50 +0000
commit03ccbd6be267147509a7d4b0304dc4b3828038d9 (patch)
tree65b5bd165020164456cf18695bdce76973d5f33f /libs/gui/LayerState.cpp
parent5bc9e109ad0b107d7f6c92f4b8aa9a560f49b686 (diff)
SF: Update InputFlinger outside main thread
Avoids parceling data inside main thread and hot path. Also avoids any binder contention with one way binder calls. See bug for more details. Re landing with the following changes: Don't discard input commands if input dispatcher is not available. This can happen if system server crashes and recovers. We want to keep input commands until input dispatcher becomes available. Bug: 206380308 Test: presubmit Test: systrace Test: flickertests Test: health/microbench/display/microbench-test-suite Change-Id: If0344abdeb28c8a409eb2f641c1e78d53e6f59b2
Diffstat (limited to 'libs/gui/LayerState.cpp')
-rw-r--r--libs/gui/LayerState.cpp4
1 files changed, 1 insertions, 3 deletions
diff --git a/libs/gui/LayerState.cpp b/libs/gui/LayerState.cpp
index cd1c810a29..ec0573a1a9 100644
--- a/libs/gui/LayerState.cpp
+++ b/libs/gui/LayerState.cpp
@@ -550,9 +550,7 @@ bool InputWindowCommands::merge(const InputWindowCommands& other) {
}
bool InputWindowCommands::empty() const {
- bool empty = true;
- empty = focusRequests.empty() && !syncInputWindows;
- return empty;
+ return focusRequests.empty() && !syncInputWindows;
}
void InputWindowCommands::clear() {