[Ozone-GBM] Pumb DriWindowDelegate throughout the platform

ui/ozone/platform/dri/gbm_surface.cc

 // 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.
 
 #include "ui/ozone/platform/dri/gbm_surface.h"
 
 #include <gbm.h>
 
 #include "base/logging.h"
 #include "ui/ozone/platform/dri/dri_buffer.h"
+#include "ui/ozone/platform/dri/dri_window_delegate.h"
 #include "ui/ozone/platform/dri/dri_wrapper.h"
 #include "ui/ozone/platform/dri/gbm_buffer_base.h"
+#include "ui/ozone/platform/dri/hardware_display_controller.h"
 #include "ui/ozone/platform/dri/scanout_buffer.h"
 
 namespace ui {
 
 namespace {
 
 class GbmSurfaceBuffer : public GbmBufferBase {
  public:
   static scoped_refptr<GbmSurfaceBuffer> CreateBuffer(DriWrapper* dri,
                                                       gbm_bo* buffer);
@@ skipping 43 matching lines @@
 }
 
 // static
 void GbmSurfaceBuffer::Destroy(gbm_bo* buffer, void* data) {
   GbmSurfaceBuffer* scoped_buffer = static_cast<GbmSurfaceBuffer*>(data);
   scoped_buffer->self_ = NULL;
 }
 
 }  // namespace
 
-GbmSurface::GbmSurface(
-    const base::WeakPtr<HardwareDisplayController>& controller,
-    gbm_device* device,
-    DriWrapper* dri)
-    : GbmSurfaceless(controller),
+GbmSurface::GbmSurface(DriWindowDelegate* window_delegate,
+                       gbm_device* device,
+                       DriWrapper* dri)
+    : GbmSurfaceless(window_delegate),
       gbm_device_(device),
       dri_(dri),
       native_surface_(NULL),
-      current_buffer_(NULL) {}
+      current_buffer_(NULL) {
+}
 
 GbmSurface::~GbmSurface() {
   if (current_buffer_)
     gbm_surface_release_buffer(native_surface_, current_buffer_);
 
   if (native_surface_)
     gbm_surface_destroy(native_surface_);
 }
 
 bool GbmSurface::Initialize() {
+  // If we're initializing the surface without a controller (possible on startup
+  // where the surface creation can happen before the native window delegate
+  // IPCs arrive), initialize the size to a valid value such that surface
+  // creation doesn't fail.
+  gfx::Size size(1, 1);
+  if (window_delegate_->GetController()) {
+    const drmModeModeInfo& mode = window_delegate_->GetController()->get_mode();
+    size.SetSize(mode.hdisplay, mode.vdisplay);
+  }
   // TODO(dnicoara) Check underlying system support for pixel format.
-  native_surface_ = gbm_surface_create(
-      gbm_device_,
-      controller_->get_mode().hdisplay,
-      controller_->get_mode().vdisplay,
-      GBM_BO_FORMAT_XRGB8888,
-      GBM_BO_USE_SCANOUT | GBM_BO_USE_RENDERING);
+  native_surface_ =
+      gbm_surface_create(gbm_device_,
+                         size.width(),
+                         size.height(),
+                         GBM_BO_FORMAT_XRGB8888,
+                         GBM_BO_USE_SCANOUT | GBM_BO_USE_RENDERING);
 
   if (!native_surface_)
     return false;
 
-  size_.SetSize(controller_->get_mode().hdisplay,
-                controller_->get_mode().vdisplay);
+  size_ = size;
   return true;
 }
 
 intptr_t GbmSurface::GetNativeWindow() {
   DCHECK(native_surface_);
   return reinterpret_cast<intptr_t>(native_surface_);
 }
 
 bool GbmSurface::ResizeNativeWindow(const gfx::Size& viewport_size) {
   if (size_ == viewport_size)
@@ skipping 10 matching lines @@
       GbmSurfaceBuffer::GetBuffer(pending_buffer);
   if (!primary) {
     primary = GbmSurfaceBuffer::CreateBuffer(dri_, pending_buffer);
     if (!primary) {
       LOG(ERROR) << "Failed to associate the buffer with the controller";
       return false;
     }
   }
 
   // The primary buffer is a special case.
-  controller_->QueueOverlayPlane(OverlayPlane(primary));
+  if (window_delegate_->GetController())
+    window_delegate_->GetController()->QueueOverlayPlane(OverlayPlane(primary));
 
   if (!GbmSurfaceless::OnSwapBuffers())
     return false;
 
   // If there was a frontbuffer, it is no longer active. Release it back to GBM.
   if (current_buffer_)
     gbm_surface_release_buffer(native_surface_, current_buffer_);
 
   current_buffer_ = pending_buffer;
   return true;
 }
 
 }  // namespace ui