Check for view focus in hideSoftInputFromView
When introducing hideSoftInputFromView in [1], we also removed the
isActive check at the call sites this replaced, to avoid calling
checkFocus again. This let through some often called code paths into
IMM, which were allowed to now create a statsToken, and start being
logged, even though they fail just a few lines after. This needlessly
fills logcat for ImeTracker logs, and skews our metrics.
This CL adds the view and window focus check at the start of
IMM#hideSoftInputFromView, before starting to log the request.
[1]: I01e5a29c0f82d068751774a2c58a1bd7b7b7f91a
Bug: 302528571
Test: observe logcat with ImeTracker tag for Launcher navigation
Change-Id: I7b305a93a7957ba5caebf49d55562fcf23f26766
1 file changed