Rendering text on insecure-site warnings for WebVR

ui/gfx/BUILD.gn

 # 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.
 
 import("//build/config/ui.gni")
 import("//device/vr/features.gni")
 import("//testing/test.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 
 if (is_android) {
@@ skipping 242 matching lines @@
     ":gfx_export",
     "//base",
     "//base:base_static",
     "//base:i18n",
     "//base/third_party/dynamic_annotations",
     "//skia",
     "//third_party/zlib",
   ]
 
   # Text rendering conditions (complicated so separated out).
-  if (use_aura || is_mac) {
+  if (use_aura || is_mac || (is_android && enable_vr)) {

[sadrul, 2017/04/13 14:57:55]

I think you can just do 'enable_vr' instead of 'is_android && enable_vr', right?

[acondor_, 2017/04/18 20:59:04]

Acknowledged.

     # Mac doesn't use RenderTextHarfBuzz by default yet.
     sources += [
       "harfbuzz_font_skia.cc",
       "harfbuzz_font_skia.h",
       "render_text_harfbuzz.cc",
       "render_text_harfbuzz.h",
       "render_text_mac.h",
       "render_text_mac.mm",
       "text_utils_skia.cc",
     ]
 
     if (!is_ios) {
       sources += [
         "render_text.cc",
         "render_text.h",
       ]
     }
   } else {
     # We don't support RenderText on these platforms.
   }
 
-  if (is_android && use_aura) {
+  if (is_android && (use_aura || enable_vr)) {

[sadrul, 2017/04/13 14:57:55]

As mthiesse@ suggested, you can get rid of the use_aura check here and below
(~309)

[acondor_, 2017/04/18 20:59:04]

Done.

     sources -= [
       "platform_font_android.cc",
       "text_utils_android.cc",
     ]
     set_sources_assignment_filter([])
     sources += [ "platform_font_linux.cc" ]
     set_sources_assignment_filter(sources_assignment_filter)
   }
 
   # iOS.
   if (is_ios) {
     set_sources_assignment_filter([])
     sources += [ "scoped_cg_context_save_gstate_mac.h" ]
     set_sources_assignment_filter(sources_assignment_filter)
   } else {
     deps += [
       "//cc/paint",
       "//third_party/harfbuzz-ng",
     ]
   }
 
   # Android.
   if (is_android) {
-    if (use_aura) {
+    if (use_aura || enable_vr) {
       sources -= [ "canvas_notimplemented.cc" ]
       sources += [ "font_fallback_android.cc" ]
     } else {
       sources -= [
         "canvas_skia.cc",
         "path.cc",
         "selection_model.cc",
       ]
     }
 
@@ skipping 443 matching lines @@
 
 fuzzer_test("color_transform_fuzztest") {
   sources = [
     "color_transform_fuzzer.cc",
   ]
   deps = [
     ":gfx",
   ]
   libfuzzer_options = [ "max_len=1024" ]
 }