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
Loading
Please register or sign in to comment