PlatformScreenOzone using DisplayManager

ui/display/manager/display_manager.h

 // Copyright (c) 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 UI_DISPLAY_MANAGER_DISPLAY_MANAGER_H_
 #define UI_DISPLAY_MANAGER_DISPLAY_MANAGER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 90 matching lines @@
   DisplayLayoutStore* layout_store() { return layout_store_.get(); }
 
   void set_delegate(Delegate* delegate) { delegate_ = delegate; }
 
   // When set to true, the DisplayManager calls OnDisplayMetricsChanged even if
   // the display's bounds didn't change. Used to swap primary display.
   void set_force_bounds_changed(bool force_bounds_changed) {
     force_bounds_changed_ = force_bounds_changed;
   }
 
+  // When set to true, DisplayManager will use DisplayConfigurator to configure

[msw, 2016/12/19 20:02:52]

nit: It's nice to have one clear comment on the member and omit a comment
altogether (avoid duplication) on the simple accessor, and it's nice when the
accessor order matches the member ordering, but I see that's neither pattern is
really employed here, so I guess it's fine as-is.

[kylechar, 2016/12/20 15:19:46]

Good point. Odds are both the setter/member variable comments won't get updated
in sync. Removed the setter comment.

+  // displays.
+  void set_configure_displays(bool configure_displays) {
+    configure_displays_ = configure_displays;
+  }
+
   // Returns the display id of the first display in the outupt list.
   int64_t first_display_id() const { return first_display_id_; }
 
   // Initializes displays using command line flag. Returns false if no command
   // line flag was provided.
   bool InitFromCommandLine();
 
   // Initialize default display.
   void InitDefaultDisplay();
 
@@ skipping 286 matching lines @@
   void RunPendingTasksForTest();
 
   // Applies the |layout| and updates the bounds of displays in |display_list|.
   // |updated_ids| contains the ids for displays whose bounds have changed.
   void ApplyDisplayLayout(const DisplayLayout& layout,
                           Displays* display_list,
                           std::vector<int64_t>* updated_ids);
 
   Delegate* delegate_ = nullptr;  // not owned.
 
+  // When set to true, DisplayManager will use DisplayConfigurator to configure
+  // displays. By default, this is set to true when running on device and false
+  // when running off device.
+  bool configure_displays_;
+
   std::unique_ptr<Screen> screen_;
 
   std::unique_ptr<DisplayLayoutStore> layout_store_;
 
   int64_t first_display_id_ = kInvalidDisplayId;
 
   // List of current active displays.
   Displays active_display_list_;
   // This list does not include the displays that will be removed if
   // |UpdateDisplaysWith| is under execution.
@@ skipping 38 matching lines @@
   base::ObserverList<DisplayObserver> observers_;
 
   base::WeakPtrFactory<DisplayManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DisplayManager);
 };
 
 }  // namespace display
 
 #endif  // UI_DISPLAY_MANAGER_DISPLAY_MANAGER_H_