Make ozone/drm/mojo more immune to startup races

ui/ozone/platform/drm/gpu/drm_thread.cc

 // Copyright 2015 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/drm/gpu/drm_thread.h"
 
 #include <gbm.h>
 #include <utility>
 
 #include "base/command_line.h"
@@ skipping 189 matching lines @@
 
 void DrmThread::SetWindowBounds(gfx::AcceleratedWidget widget,
                                 const gfx::Rect& bounds) {
   screen_manager_->GetWindow(widget)->SetBounds(bounds);
 }
 
 void DrmThread::SetCursor(const gfx::AcceleratedWidget& widget,
                           const std::vector<SkBitmap>& bitmaps,
                           const gfx::Point& location,
                           int32_t frame_delay_ms) {
-  screen_manager_->GetWindow(widget)
-      ->SetCursor(bitmaps, location, frame_delay_ms);
+  auto window = screen_manager_->GetWindow(widget);

[sadrul, 2017/05/29 16:50:15]

I think we suggest using 'auto*' here.

+  // TODO(rjkroege): This check is needed only to address mushrome startup

[dnicoara, 2017/05/29 17:01:54]

Why isn't the |drm_thread_| check enough for this case?

I'm more concerned by MoveCursor() since that skips the UI thread.

Wouldn't all window operations need this check?

+  // synchronization issues. Remove as part of http://crbug.com/581462
+  if (!window)
+    return;
+  window->SetCursor(bitmaps, location, frame_delay_ms);
 }
 
 void DrmThread::MoveCursor(const gfx::AcceleratedWidget& widget,
                            const gfx::Point& location) {
   screen_manager_->GetWindow(widget)->MoveCursor(location);
 }
 
 void DrmThread::CheckOverlayCapabilities(
     gfx::AcceleratedWidget widget,
     const std::vector<OverlayCheck_Params>& overlays,
@@ skipping 68 matching lines @@
                                        correction_matrix);
 }
 
 // DrmThread requires a BindingSet instead of a simple Binding because it will
 // be used from multiple threads in multiple processes.
 void DrmThread::AddBinding(ozone::mojom::DeviceCursorRequest request) {
   bindings_.AddBinding(this, std::move(request));
 }
 
 }  // namespace ui