mash: Convert all of //ash/system to use WmShelf

ash/shelf/shelf_layout_manager.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_SHELF_SHELF_LAYOUT_MANAGER_H_
 #define ASH_SHELF_SHELF_LAYOUT_MANAGER_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 192 matching lines @@
 
  private:
   class AutoHideEventFilter;
   class RootWindowControllerObserverImpl;
   class UpdateShelfObserver;
   friend class AshPopupAlignmentDelegateTest;
   friend class ash::ScreenAsh;
   friend class PanelLayoutManagerTest;
   friend class ShelfLayoutManagerTest;
   friend class ToastManagerTest;
+  friend class WmShelfAura;

[James Cook, 2016/06/24 23:20:08]

For OnKeyboardBoundsChanging(), and I suspect a few more things in the future.

[msw, 2016/06/25 00:18:29]

Maybe move that to public or something like shelf_test_api.h?
Maybe tests should call KeyboardController::NotifyKeyboardBoundsChanging?

[James Cook, 2016/06/25 01:20:58]

Moved it to public and reordered definition in .cc. I didn't use
shelf_test_api.h because I didn't want to worry about GN's testonly code vs.
ForTesting() in WmShelf. Maybe in the future we can create a WmShelfTestApi if
WmShelf grows too many test methods.

As mentioned offline I didn't have much luck in making the test use
KeyboardController directly.

   FRIEND_TEST_ALL_PREFIXES(ash::AshPopupAlignmentDelegateTest, AutoHide);
   FRIEND_TEST_ALL_PREFIXES(ash::WebNotificationTrayTest, PopupAndFullscreen);
 
   struct TargetBounds {
     TargetBounds();
     ~TargetBounds();
 
     float opacity;
     float status_opacity;
     gfx::Rect shelf_bounds_in_root;
@@ skipping 172 matching lines @@
 
   std::unique_ptr<RootWindowControllerObserverImpl>
       root_window_controller_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ShelfLayoutManager);
 };
 
 }  // namespace ash
 
 #endif  // ASH_SHELF_SHELF_LAYOUT_MANAGER_H_