summaryrefslogtreecommitdiff
path: root/libs/gui/DisplayLuts.cpp
diff options
context:
space:
mode:
author Siarhei Vishniakou <svv@google.com> 2023-12-18 20:41:08 -0800
committer Siarhei Vishniakou <svv@google.com> 2023-12-28 10:12:51 -0800
commitd3061ab7049fd1bfe3daf7fd89003e0b22f2e11c (patch)
treeecb420b8bbb7dd8f27cf1105a45fd83157640d7a /libs/gui/DisplayLuts.cpp
parent9d3866046b77934acf4fcf30f685c81f1c9bfeca (diff)
Use dynamic event factory in dispatcher tests
This factory allows us to obtain ownership of the returned events. At the same time, remove LOG(FATAL) from the tests. Technically, the new factory approach isn't necessary. We could have simply switched to returning raw unowned pointers instead of references, and added the null check like in this CL. This would have been sufficient to remove the fatal log statements. However, since this code is already being touched here, might as well improve it further by changing to the owned pointer model. Without LOG(FATAL) in the tests, tests are allowed to run to completion when there's a bug in the code. This allows the developer to understand how many tests are failing, thus providing better signal about the work in progress CL. Bug: 308531018 Test: TEST=inputflinger_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST Change-Id: Iee88b35e871ef391aac0d68aa8322486287524e9
Diffstat (limited to 'libs/gui/DisplayLuts.cpp')
0 files changed, 0 insertions, 0 deletions