[WIP] Make content_shell run under Wayland

ui/ozone/platform/wayland/ozone_platform_wayland.cc

 // Copyright 2016 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.
 
 #include "ui/ozone/platform/wayland/ozone_platform_wayland.h"
 
 #include "base/memory/ptr_util.h"
 #include "ui/ozone/common/native_display_delegate_ozone.h"
 #include "ui/ozone/common/stub_overlay_manager.h"
 #include "ui/ozone/platform/wayland/wayland_display.h"
 #include "ui/ozone/platform/wayland/wayland_surface_factory.h"
 #include "ui/ozone/platform/wayland/wayland_window.h"
 #include "ui/ozone/public/cursor_factory_ozone.h"
 #include "ui/ozone/public/gpu_platform_support.h"
 #include "ui/ozone/public/gpu_platform_support_host.h"
 #include "ui/ozone/public/input_controller.h"
 #include "ui/ozone/public/ozone_platform.h"
 #include "ui/ozone/public/system_input_injector.h"
+#include "ui/views/widget/desktop_aura/desktop_factory_wayland.h"
 
 namespace ui {
 
 namespace {
 
 class OzonePlatformWayland : public OzonePlatform {
  public:
   OzonePlatformWayland() {}
   ~OzonePlatformWayland() override {}
 
@@ skipping 44 matching lines @@
   void InitializeUI() override {
     display_.reset(new WaylandDisplay);
     if (!display_->Initialize())
       LOG(FATAL) << "Failed to initialize Wayland platform";
 
     cursor_factory_.reset(new CursorFactoryOzone);
     overlay_manager_.reset(new StubOverlayManager);
     input_controller_ = CreateStubInputController();
     surface_factory_.reset(new WaylandSurfaceFactory(display_.get()));
     gpu_platform_support_host_.reset(CreateStubGpuPlatformSupportHost());
+    desktop_factory_.reset(new views::DesktopFactoryOzoneWayland());

[tonikitoo, 2016/06/06 15:29:59]

I joone. For the record, this is the biggest goal of
https://codereview.chromium.org/2024953007/ .

Ideally, DesktopFactoryOzone{Platform} should be picked up at runtime, in
accordance to --ozone-platform={platform} .. OR the default platform, if none is
specified at runtime.

+    views::DesktopFactoryOzone::SetInstance(desktop_factory_.get());

[tonikitoo, 2016/06/06 15:29:59]

Also, https://codereview.chromium.org/2024953007/ eliminates SetInstance making
the DesktopFactoryOzone{Bleh} instance a known leaky object. This is a similar
pattern than OzonePlatform{Bleh}.

What do you think of this design?

[joone, 2016/06/06 18:22:22]

Great.

   }
 
   void InitializeGPU() override {
     // Don't reinitialize the surface factory in case InitializeUI was called
     // previously in the same process.
     if (!surface_factory_)
       surface_factory_.reset(new WaylandSurfaceFactory(nullptr));
     gpu_platform_support_.reset(CreateStubGpuPlatformSupport());
   }
 
  private:
   std::unique_ptr<WaylandDisplay> display_;
   std::unique_ptr<WaylandSurfaceFactory> surface_factory_;
+  std::unique_ptr<views::DesktopFactoryOzone> desktop_factory_;

[tonikitoo, 2016/06/06 15:29:59]

Ideally ozone/platform should not depend on ui/views/widget. That was my main
motivation in the https://codereview.chromium.org/2024953007/.

[joone, 2016/06/06 18:22:21]

Yes, I like your approach.

   std::unique_ptr<CursorFactoryOzone> cursor_factory_;
   std::unique_ptr<StubOverlayManager> overlay_manager_;
   std::unique_ptr<InputController> input_controller_;
   std::unique_ptr<GpuPlatformSupportHost> gpu_platform_support_host_;
   std::unique_ptr<GpuPlatformSupport> gpu_platform_support_;
 
   DISALLOW_COPY_AND_ASSIGN(OzonePlatformWayland);
 };
 
 }  // namespace
 
 OzonePlatform* CreateOzonePlatformWayland() {
   return new OzonePlatformWayland;
 }
 
 }  // namespace ui