Support offscreen contexts which own their backing surface

chrome/browser/about_flags.cc

Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 855e4aeb2525d3ee4ec4a44977032db4c0971cd0..a01de4d6ba0fe286faf318e4c9c26add0e16d311 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -1612,6 +1612,10 @@ const FeatureEntry kFeatureEntries[] = {
     {"enable-webvr", IDS_FLAGS_WEBVR_NAME, IDS_FLAGS_WEBVR_DESCRIPTION, kOsAll,
      SINGLE_VALUE_TYPE(switches::kEnableWebVR)},
 #if defined(ENABLE_WEBVR)
+    {"enable-webvr-experimental-rendering",
+     IDS_FLAGS_WEBVR_EXPERIMENTAL_RENDERING_NAME,
+     IDS_FLAGS_WEBVR_EXPERIMENTAL_RENDERING_DESCRIPTION, kOsAll,

[amp, 2017/02/15 17:44:03]

Should we limit this to Android only for now?

[klausw, 2017/02/15 19:47:22]

Done. I assume that kOsAndroid here means to just keep it auto-disabled and
hidden on other platforms? An ifdef wouldn't work since the feature check is in
non-Android-specific code.

[amp, 2017/02/15 20:04:22]

I'm not actually sure what the os param does.  Maybe it grays it out on other
platforms (but we are hiding it behind a build flag so it won't show up anyway).

Either looks good to keep the os param in sync with VrShell below it.

+     SINGLE_VALUE_TYPE(switches::kEnableWebVRExperimentalRendering)},

[amp, 2017/02/15 17:44:03]

This should be referring to the feature (like vr shell does below), not to the
switch (which aren't needed).

[klausw, 2017/02/15 19:47:22]

Done.

     {"enable-vr-shell", IDS_FLAGS_ENABLE_VR_SHELL_NAME,
      IDS_FLAGS_ENABLE_VR_SHELL_DESCRIPTION, kOsAndroid,
      FEATURE_VALUE_TYPE(features::kVrShell)},