Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1224)

Unified Diff: services/ui/display/platform_screen_ozone.h

Issue 2415163002: Expand and split DisplayController mojom. (Closed)
Patch Set: Actually fix case statements. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: services/ui/display/platform_screen_ozone.h
diff --git a/services/ui/display/platform_screen_ozone.h b/services/ui/display/platform_screen_ozone.h
index bcdbdcb4e6a0c1026ab4a76675f380c656352496..b60b8e6bcd8425b8f05280566ebc26719ff851bf 100644
--- a/services/ui/display/platform_screen_ozone.h
+++ b/services/ui/display/platform_screen_ozone.h
@@ -17,6 +17,7 @@
#include "services/service_manager/public/cpp/interface_factory.h"
#include "services/ui/display/platform_screen.h"
#include "services/ui/public/interfaces/display/display_controller.mojom.h"
+#include "services/ui/public/interfaces/display/test_display_controller.mojom.h"
#include "ui/display/chromeos/display_configurator.h"
#include "ui/display/display.h"
#include "ui/display/types/fake_display_controller.h"
@@ -29,7 +30,9 @@ class PlatformScreenOzone
: public PlatformScreen,
public ui::DisplayConfigurator::Observer,
public service_manager::InterfaceFactory<mojom::DisplayController>,
- public mojom::DisplayController {
+ public service_manager::InterfaceFactory<mojom::TestDisplayController>,
+ public mojom::DisplayController,
+ public mojom::TestDisplayController {
public:
PlatformScreenOzone();
~PlatformScreenOzone() override;
@@ -40,8 +43,14 @@ class PlatformScreenOzone
void RequestCloseDisplay(int64_t display_id) override;
int64_t GetPrimaryDisplayId() const override;
+ // mojom::TestDisplayController:
+ void ToggleAddRemoveDisplay() override;
+
// mojom::DisplayController:
- void ToggleVirtualDisplay() override;
+ void SwapPrimaryDisplay() override;
+ void SetDisplayWorkArea(int64_t display_id,
+ const gfx::Size& size,
+ const gfx::Insets& insets) override;
private:
// TODO(kylechar): This struct is just temporary until we migrate
@@ -105,6 +114,10 @@ class PlatformScreenOzone
void Create(const service_manager::Identity& remote_identity,
mojom::DisplayControllerRequest request) override;
+ // mojo::InterfaceFactory<mojom::TestDisplayController>:
+ void Create(const service_manager::Identity& remote_identity,
+ mojom::TestDisplayControllerRequest request) override;
+
ui::DisplayConfigurator display_configurator_;
PlatformScreenDelegate* delegate_ = nullptr;
@@ -121,7 +134,8 @@ class PlatformScreenOzone
std::vector<DisplayInfo> cached_displays_;
gfx::Point next_display_origin_;
- mojo::BindingSet<mojom::DisplayController> bindings_;
+ mojo::BindingSet<mojom::DisplayController> controller_bindings_;
+ mojo::BindingSet<mojom::TestDisplayController> test_bindings_;
DISALLOW_COPY_AND_ASSIGN(PlatformScreenOzone);
};
« no previous file with comments | « ash/mus/accelerators/accelerator_controller_delegate_mus.cc ('k') | services/ui/display/platform_screen_ozone.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698