cros: Add TODO to remove deps for Shortcuts Viewer

This cl adds TODO to remove the dependency on aura::Window in Keyboard
Shortcuts Viewer. In addition, this cl removes unused deps and include.

Bug: 833673
Test: Manual
Change-Id: Icc4fd0726cace1971674e371bb5053b20575933b
Reviewed-on: https://chromium-review.googlesource.com/1014663Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Tao Wu <wutao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551348}

chrome/browser/ui/ash/ksv/DEPS

@@ -7,7 +7,6 @@ include_rules = [
   "+ash/components/shortcut_viewer",
   "+ash/public",
   "+ash/shell.h",
-  "+ash/wm/window_util.h",
   "+chrome/browser/ui/ash/ksv",
 ]
 

chrome/browser/ui/ash/ksv/keyboard_shortcut_viewer_util.cc

@@ -6,11 +6,11 @@
 
 #include "ash/components/shortcut_viewer/views/keyboard_shortcut_view.h"
 #include "ash/shell.h"
-#include "ash/wm/window_util.h"
 
 namespace keyboard_shortcut_viewer_util {
 
 void ShowKeyboardShortcutViewer() {
+  // TODO(https://crbug.com/833673): Remove the dependency on aura::Window.
   keyboard_shortcut_viewer::KeyboardShortcutView::Show(
       ash::Shell::GetRootWindowForNewWindows());
 }