Remove DisplayChangeObserver ash dependencies.

ash/display/display_change_observer_chromeos.h

 // Copyright 2013 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_DISPLAY_DISPLAY_CHANGE_OBSERVER_CHROMEOS_H_
 #define ASH_DISPLAY_DISPLAY_CHANGE_OBSERVER_CHROMEOS_H_
 
 #include <stdint.h>
 
+#include <memory>
 #include <vector>
 
 #include "ash/ash_export.h"
-#include "ash/common/shell_observer.h"
 #include "base/macros.h"
 #include "ui/display/manager/chromeos/display_configurator.h"
 #include "ui/display/manager/managed_display_info.h"
 #include "ui/events/devices/input_device_event_observer.h"
 
+namespace display {
+class DisplayManager;
+}
+
 namespace ash {
 
 class DisplaySnapshot;
 
-// An object that observes changes in display configuration and
-// update DisplayManagers.
+// An object that observes changes in display configuration and updates
+// DisplayManager.
 class DisplayChangeObserver : public ui::DisplayConfigurator::StateController,
                               public ui::DisplayConfigurator::Observer,
-                              public ui::InputDeviceEventObserver,
-                              public ShellObserver {
+                              public ui::InputDeviceEventObserver {
  public:
   // Returns the mode list for internal display.
   ASH_EXPORT static display::ManagedDisplayInfo::ManagedDisplayModeList
   GetInternalManagedDisplayModeList(
       const display::ManagedDisplayInfo& display_info,
       const ui::DisplaySnapshot& output);
 
   // Returns the resolution list.
   ASH_EXPORT static display::ManagedDisplayInfo::ManagedDisplayModeList
   GetExternalManagedDisplayModeList(const ui::DisplaySnapshot& output);
 
   DisplayChangeObserver();
   ~DisplayChangeObserver() override;
 
+  void set_display_manager(display::DisplayManager* display_manager) {
+    display_manager_ = display_manager;
+  }
+
+  void set_display_configurator(ui::DisplayConfigurator* display_configurator) {
+    display_configurator_ = display_configurator;
+  }
+
   // ui::DisplayConfigurator::StateController overrides:
   ui::MultipleDisplayState GetStateForDisplayIds(
       const ui::DisplayConfigurator::DisplayStateList& outputs) const override;
   bool GetResolutionForDisplayId(int64_t display_id,
                                  gfx::Size* size) const override;
 
   // Overriden from ui::DisplayConfigurator::Observer:
   void OnDisplayModeChanged(
       const ui::DisplayConfigurator::DisplayStateList& outputs) override;
   void OnDisplayModeChangeFailed(
       const ui::DisplayConfigurator::DisplayStateList& displays,
       ui::MultipleDisplayState failed_new_state) override;
 
   // Overriden from ui::InputDeviceEventObserver:
   void OnTouchscreenDeviceConfigurationChanged() override;
 
-  // Overriden from ShellObserver:
-  void OnAppTerminating() override;
-
   // Exposed for testing.
   ASH_EXPORT static float FindDeviceScaleFactor(float dpi);
 
  private:

[oshima, 2016/12/02 19:07:45]

can you document ownership/lifetime of these objects?

[kylechar, 2016/12/02 20:32:36]

Done.

+  display::DisplayManager* display_manager_ = nullptr;
+  ui::DisplayConfigurator* display_configurator_ = nullptr;
+
   DISALLOW_COPY_AND_ASSIGN(DisplayChangeObserver);
 };
 
 }  // namespace ash
 
 #endif  // ASH_DISPLAY_DISPLAY_CHANGE_OBSERVER_CHROMEOS_H_