Remove Sync GetPose VRService call, implement VRVSyncProvider

content/browser/BUILD.gn

Index: content/browser/BUILD.gn
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index 826786abb707a986b1f58782d42d0b5e4a14d56e..85712c3f552ab21bc3c647b16b37498db5fb3d66 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -1994,11 +1994,7 @@ source_set("browser") {
     deps += [ "//ui/ozone" ]
   }
 
-  if (enable_webvr) {

[clamy, 2017/01/25 15:19:47]

If I understand correctly this means we'll always build vr by default, right?
Why the change?

[mthiesse, 2017/01/25 15:36:13]

To be honest, I'm really not sure how this managed to link at all before my
change here, as render_frame_host_impl includes device/vr/vr_service_impl.h
unconditionally.

The change to device/vr/BUILD.gn that adds DEVICE_VR_EXPORT to the VRService
mojo files somehow exposed this and prevented render_frame_host_impl from
linking.

Most of VR still won't be built by default, this just builds the
platform-independent mojo bindings by default.

-    deps += [ "//device/vr" ]
-  } else {
-    deps += [ "//device/vr:mojo_bindings" ]
-  }
+  deps += [ "//device/vr" ]
 
   if (enable_ipc_fuzzer) {
     configs += [ "//tools/ipc_fuzzer:ipc_fuzzer_config" ]