Enable WebVR origin trial experiments.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index c8b5dcc9fbd8a11090cd2d404fdbcc9cc96cc1bd..7423a935902d8cc88bb6d5997bee0ad85cc1a633 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -2201,13 +2201,8 @@ void RenderFrameHostImpl::RegisterMojoInterfaces() {
                  routing_id_));
 
 #if defined(ENABLE_WEBVR)
-  const base::CommandLine& browser_command_line =
-      *base::CommandLine::ForCurrentProcess();
-
-  if (browser_command_line.HasSwitch(switches::kEnableWebVR)) {

[clamy, 2016/10/07 11:09:54]

Why not use directly "if
(base::CommandLine::ForCurrentProcess()->HasSwitch(switches::kEnableWebVR))"?
This is what's done everywhere in the code.

[iclelland, 2016/10/07 14:51:50]

Isn't that line being deleted, though?

[clamy, 2016/10/07 14:54:40]

I'm not quite sure what you mean? If you like at
https://cs.chromium.org/chromium/src/content/common/site_isolation_policy.cc?...
for example, this is what we use. Note that the functions are used all over the
code, at various points in time.

[iclelland, 2016/10/07 15:11:16]

I mean that this CL *deletes* the line you're talking about.

base::CommandLine::ForCurrentProcess()->HasSwitch(...) is definitely the idiom
of choice, but it's no longer required once this feature is enabled as an origin
trial.

[clamy, 2016/10/07 15:13:59]

Sorry my bad, I read too quickly :-/.

-    GetInterfaceRegistry()->AddInterface<device::VRService>(
-        base::Bind(&device::VRServiceImpl::BindRequest));
-  }
+  GetInterfaceRegistry()->AddInterface<device::VRService>(
+      base::Bind(&device::VRServiceImpl::BindRequest));
 #endif
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableGenericSensors)) {