Remove Sync GetPose VRService call, implement VRVSyncProvider

content/browser/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/features.gni")
 import("//build/config/ui.gni")
 import("//media/media_options.gni")
 import("//ppapi/features/features.gni")
 import("//printing/features/features.gni")
 import("//third_party/WebKit/public/public_features.gni")
@@ skipping 1976 matching lines @@
   }
 
   if (enable_mojo_media) {
     configs += [ "//media/mojo/services:mojo_media_config" ]
   }
 
   if (use_ozone) {
     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" ]
   }
 
   if (use_minikin_hyphenation) {
     sources += [
       "hyphenation/hyphenation_impl.cc",
       "hyphenation/hyphenation_impl.h",
     ]
   }
 }
 
 # See comment at the top of //content/BUILD.gn for how this works.
 group("for_content_tests") {
   visibility = [ "//content/test/*" ]
   if (!is_component_build) {
     public_deps = [
       ":browser",
     ]
   } else {
     public_deps = [
       "//third_party/leveldatabase",
     ]
   }
 }