Hook up insecure content warnings for http webVR presentation.

chrome/browser/android/vr_shell/ui_scene.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/ui_scene.h"
 
 #include <string>
 #include <utility>
 
 #include "base/memory/ptr_util.h"
@@ skipping 44 matching lines @@
 
 void UiScene::AddUiElement(std::unique_ptr<UiElement> element) {
   CHECK_GE(element->id, 0);
   CHECK_EQ(GetUiElementById(element->id), nullptr);
   if (element->parent_id >= 0) {
     CHECK_NE(GetUiElementById(element->parent_id), nullptr);
   } else {
     CHECK_EQ(element->x_anchoring, XAnchoring::XNONE);
     CHECK_EQ(element->y_anchoring, YAnchoring::YNONE);
   }
+  if (gl_initialized_)
+    element->Initialize();
   ui_elements_.push_back(std::move(element));
 }
 
 void UiScene::RemoveUiElement(int element_id) {
   for (auto it = ui_elements_.begin(); it != ui_elements_.end(); ++it) {
     if ((*it)->id == element_id) {
       if ((*it)->fill == Fill::CONTENT) {
         content_element_ = nullptr;
       }
       ui_elements_.erase(it);
@@ skipping 121 matching lines @@
 
     element->computed_opacity *= parent->opacity;
     element->computed_lock_to_fov = parent->lock_to_fov;
   }
 
   vr::MatrixMul(inheritable->to_world, transform->to_world,
                 &transform->to_world);
   element->dirty = false;
 }
 
+void UiScene::OnGLInitialized() {
+  gl_initialized_ = true;
+  for (const auto& element : ui_elements_) {

[cjgrant, 2017/04/21 14:30:13]

This is another situation where the const should be breaking compilation but
isn't.

[mthiesse, 2017/04/21 15:05:12]

No, because the unique_ptr is constant, but -> doesn't return a const pointer,
even when the unique_ptr is const.

[cjgrant, 2017/04/21 15:19:33]

Right.  I mean that we shouldn't bother with the const here, as it looks
confusing.

[mthiesse, 2017/04/21 17:17:30]

Fixed by getting rid of auto so it's not confusing.

+    element->Initialize();
+  }
+}
+
 }  // namespace vr_shell