Add support for virtual displays

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 <queue>
 #include <string>
 #include <vector>
 
 #include "base/event_types.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
 #include "base/memory/weak_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/display_snapshot_virtual.h"
 #include "ui/display/chromeos/query_content_protection_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 231 matching lines @@
       int64_t display_id);
 
   // Updates the color calibration to |new_profile|.
   bool SetColorCalibrationProfile(int64_t display_id,
                                   ui::ColorCalibrationProfile new_profile);
 
   // Sets the gamma ramp for |display_id| to the values in |lut|.
   bool SetGammaRamp(int64_t display_id,
                     const std::vector<GammaRampRGBEntry>& lut);
 
+  // Enables/disables virtual display.
+  int64_t AddVirtualDisplay(gfx::Size display_size);
+  bool RemoveVirtualDisplay(int64_t display_id);
+
  private:
   class DisplayLayoutManagerImpl;
 
   // Mapping a client to its protection request.
   typedef std::map<ContentProtectionClientId, ContentProtections>
       ProtectionRequests;
 
   // Performs platform specific delegate initialization.
   scoped_ptr<NativeDisplayDelegate> CreatePlatformNativeDisplayDelegate();
 
@@ skipping 132 matching lines @@
   // Display controlled by an external entity.
   bool display_externally_controlled_;
 
   // True if a TakeControl or RelinquishControl has been called but the response
   // hasn't arrived yet.
   bool display_control_changing_;
 
   // Whether the displays are currently suspended.
   bool displays_suspended_;
 
+  // Virtual display control.
+  ScopedVector<DisplaySnapshot> virtual_display_snapshots_;
+
+  // Last used virtual display id.
+  int64_t last_virtual_display_id_ = 0;

[oshima, 2015/11/24 22:19:27]

Can you factor out GetID in ui/display/util/edid_parser.cc and use
0x8000 as manufacturer id, and 0x as product code and this id as a output index?
please make sure that it will not overflow.

[robert.bradford, 2015/11/26 16:29:40]

Ah, that's a great solution. I wasn't very happy with it as it stood.

+
   scoped_ptr<DisplayLayoutManager> layout_manager_;
 
   scoped_ptr<UpdateDisplayConfigurationTask> configuration_task_;
 
   // This must be the last variable.
   base::WeakPtrFactory<DisplayConfigurator> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DisplayConfigurator);
 };
 
 }  // namespace ui
 
 #endif  // UI_DISPLAY_CHROMEOS_DISPLAY_CONFIGURATOR_H_