Add display task to trigger display configuration

ui/display/chromeos/display_configurator.h

 // Copyright 2014 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 UI_DISPLAY_CHROMEOS_DISPLAY_CONFIGURATOR_H_
 #define UI_DISPLAY_CHROMEOS_DISPLAY_CONFIGURATOR_H_
 
 #include <stdint.h>
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/event_types.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/timer/timer.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
+#include "ui/display/chromeos/configure_displays_task.h"
 #include "ui/display/display_export.h"
 #include "ui/display/types/display_constants.h"
 #include "ui/display/types/native_display_observer.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace gfx {
 class Point;
 class Size;
 }
 
@@ skipping 58 matching lines @@
   // Interface for classes that implement software based mirroring.
   class SoftwareMirroringController {
    public:
     virtual ~SoftwareMirroringController() {}
 
     // Called when the hardware mirroring failed.
     virtual void SetSoftwareMirroring(bool enabled) = 0;
     virtual bool SoftwareMirroringEnabled() const = 0;
   };
 
+  class DisplayLayoutManager {
+   public:
+    virtual ~DisplayLayoutManager() {}
+
+    virtual SoftwareMirroringController* GetSoftwareMirroringController()
+        const = 0;
+
+    virtual StateController* GetStateController() const = 0;
+
+    // Returns the current display state.
+    virtual MultipleDisplayState GetDisplayState() const = 0;
+
+    // Returns the current power state.
+    virtual chromeos::DisplayPowerState GetPowerState() const = 0;
+
+    // Parses the displays and adds |mirror_mode| and |selected_mode|.

[Daniel Erat, 2014/12/11 23:34:07]

the mirror_mode and selected_mode mentions seem confusing; there aren't any
arguments or members with those names. are those fields within DisplayState? if
so, i'd probably say something like "... and sets mirrored and selected modes
within the return DisplayStates." instead

+    // TODO(dnicoara): This operation doesn't depend on state and could be done
+    // directly in |GetDisplayLayout()|. Though I need to make sure that there
+    // are no uses for those fields outside DisplayConfigurator.
+    virtual std::vector<DisplayState> ParseDisplays(
+        const std::vector<DisplaySnapshot*>& displays) const = 0;
+
+    // Based on the given |displays|, display state and power state, it will
+    // create display configuration requests which will then be used to
+    // configure the hardware. The requested configuration is stored in
+    // |requests| and |framebuffer_size|.
+    virtual bool GetDisplayLayout(
+        const std::vector<DisplayState>& displays,
+        MultipleDisplayState new_display_state,
+        chromeos::DisplayPowerState new_power_state,
+        std::vector<DisplayConfigureRequest>* requests,
+        gfx::Size* framebuffer_size) const = 0;
+  };
+
   // Helper class used by tests.
   class TestApi {
    public:
     TestApi(DisplayConfigurator* configurator) : configurator_(configurator) {}
     ~TestApi() {}
 
     // If |configure_timer_| is started, stops the timer, runs
     // ConfigureDisplays(), and returns true; returns false otherwise.
     bool TriggerConfigureTimeout() WARN_UNUSED_RESULT;
 
@@ skipping 253 matching lines @@
 
   // Display controlled by an external entity.
   bool display_externally_controlled_;
 
   DISALLOW_COPY_AND_ASSIGN(DisplayConfigurator);
 };
 
 }  // namespace ui
 
 #endif  // UI_DISPLAY_CHROMEOS_DISPLAY_CONFIGURATOR_H_