GitLab

The present CL addresses the problem that AccessibilityControllerImpl
never actually removes itself as an observer of TabletModeController. In
the following part of the destructor, the "if" condition is always false
because the tablet mode controller is destroyed first:

  if (Shell::Get()->tablet_mode_controller())
    Shell::Get()->tablet_mode_controller()->RemoveObserver(this);

Change-Id: Ib35b19751862921691c767520e6803007c763a8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816906Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Avery Musbach <amusbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#698624}