Move Ozone GPU initialization.

gpu/ipc/service/gpu_init.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 "gpu/ipc/service/gpu_init.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/third_party/dynamic_annotations/dynamic_annotations.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/trace_event/trace_event.h"
 #include "gpu/command_buffer/service/gpu_switches.h"
 #include "gpu/config/gpu_driver_bug_list.h"
 #include "gpu/config/gpu_info_collector.h"
 #include "gpu/config/gpu_switches.h"
 #include "gpu/config/gpu_util.h"
 #include "gpu/ipc/service/gpu_watchdog_thread.h"
 #include "gpu/ipc/service/switches.h"
 #include "ui/gfx/switches.h"
 #include "ui/gl/gl_implementation.h"
 #include "ui/gl/gl_switches.h"
 #include "ui/gl/init/gl_factory.h"
 
+#if defined(USE_OZONE)
+#include "ui/ozone/public/ozone_platform.h"
+#endif
+
 namespace gpu {
 
 namespace {
 
 void GetGpuInfoFromCommandLine(gpu::GPUInfo& gpu_info,
                                const base::CommandLine& command_line) {
   if (!command_line.HasSwitch(switches::kGpuVendorID) ||
       !command_line.HasSwitch(switches::kGpuDeviceID) ||
       !command_line.HasSwitch(switches::kGpuDriverVersion))
     return;
@@ skipping 133 matching lines @@
 #if defined(OS_LINUX)
   // On Chrome OS ARM Mali, GPU driver userspace creates threads when
   // initializing a GL context, so start the sandbox early.
   if (command_line.HasSwitch(switches::kGpuSandboxStartEarly))
     gpu_info_.sandboxed =
         sandbox_helper_->EnsureSandboxInitialized(watchdog_thread_.get());
 #endif  // defined(OS_LINUX)
 
   base::TimeTicks before_initialize_one_off = base::TimeTicks::Now();
 
+#if defined(USE_OZONE)
+  // Initialize Ozone GPU after the watchdog and sandbox are started.

[piman, 2017/03/15 19:48:21]

The sandbox is generally not started at this point (early sandbox is the
exception for a couple of drivers that create threads).
I think this is still ok (since it used to be in InitializeGLOneOff right
below), but you may want to update the comment to reflect what's really
happening.

[kylechar, 2017/03/15 20:59:08]

Thanks for the clarification, I changed the comment.

+  ui::OzonePlatform::InitParams params;
+  params.single_process = false;
+  ui::OzonePlatform::InitializeForGPU(params);
+#endif
+
   // Load and initialize the GL implementation and locate the GL entry points if
   // needed. This initialization may have already happened if running in the
   // browser process, for example.
   bool gl_initialized = gl::GetGLImplementation() != gl::kGLImplementationNone;
   if (!gl_initialized)
     gl_initialized = gl::init::InitializeGLOneOff();
 
   if (!gl_initialized) {
     VLOG(1) << "gl::init::InitializeGLOneOff failed";
     return false;
@@ skipping 46 matching lines @@
     watchdog_thread_ = gpu::GpuWatchdogThread::Create();
   }
 
   if (!gpu_info_.sandboxed)
     gpu_info_.sandboxed =
         sandbox_helper_->EnsureSandboxInitialized(watchdog_thread_.get());
   return true;
 }
 
 }  // namespace gpu