PROTOTYPE (incomplete): Add quad renderer and stub bits to handle security warnings.

chrome/browser/android/vr_shell/vr_gl_thread.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 "chrome/browser/android/vr_shell/vr_gl_thread.h"
 
 #include <utility>
 
 #include "chrome/browser/android/vr_shell/ui_scene.h"
 #include "chrome/browser/android/vr_shell/ui_scene_manager.h"
@@ skipping 16 matching lines @@
       initially_web_vr_(initially_web_vr),
       reprojected_rendering_(reprojected_rendering) {}
 
 VrGLThread::~VrGLThread() {
   Stop();
 }
 
 void VrGLThread::Init() {
   scene_ = base::MakeUnique<UiScene>();
   scene_manager_ = base::MakeUnique<UiSceneManager>(scene_.get());
+
+  // TODO: Clear up the init order.  We should create the GL object first, as it

[cjgrant, 2017/04/19 16:24:52]

Elaborating:  This isn't a big deal, but we should do it correctly so that we
don't get sporadic failures on exit.

+  // is the tool used to draw the scene, and it supplies texture IDs as elements
+  // are created.  However, the GL object also needs a pointer to the scene, so
+  // it has something to draw.  Possibly just supply the scene pointer to
+  // VrShellGl after it's created.  Whatever happens, be sure to tear things
+  // down later in the inverse order.
+
   vr_shell_gl_ = base::MakeUnique<VrShellGl>(
       std::move(weak_vr_shell_), std::move(main_thread_task_runner_), gvr_api_,
       initially_web_vr_, reprojected_rendering_, scene_.get());
   weak_vr_shell_gl_ = vr_shell_gl_->GetWeakPtr();
   weak_scene_manager_ = scene_manager_->GetWeakPtr();
   vr_shell_gl_->Initialize();
 }
 
 void VrGLThread::CleanUp() {
   vr_shell_gl_.reset();
   scene_manager_.reset();
   scene_.reset();
 }
 
 }  // namespace vr_shell