GitLab

When observers are notified about the VK being enabled / disabled, the
state of the KeyboardController at the time is wrong. For example,
when notified about disabling the VK, KeyboardController::IsEnabled()
still returns true (!).

We move the observer notification code to the correct place and add
tests to verify the state of the KeyboardController at the time of
notification.

Bug: 943446
Change-Id: I663042f743826afdb9880c538e6110ca13b4a6d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572983
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#654787}