Adding Chrome-side WebVR interface

content/browser/vr/vr_device.h

+// Copyright 2015 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.
+
+#ifndef CONTENT_BROWSER_VR_VR_DEVICE_H
+#define CONTENT_BROWSER_VR_VR_DEVICE_H
+
+namespace blink {
+struct WebHMDSensorState;
+struct WebVRFieldOfView;
+class WebVRDevice;
+struct WebVRVector4;
+}
+
+namespace ui {
+class BaseWindow;
+}
+
+namespace content {
+
+class VRDeviceProvider;
+
+class VRDevice {
+ public:
+  VRDevice(VRDeviceProvider* provider);

[mdempsky, 2015/05/11 19:25:01]

Nit: Add "explicit".

+  virtual ~VRDevice();
+
+  VRDeviceProvider* provider() const { return provider_; }
+  unsigned int id() const { return id_; }
+
+  virtual void GetVRDevice(blink::WebVRDevice* device) = 0;
+  virtual void GetSensorState(blink::WebHMDSensorState* state) = 0;
+  virtual void ResetSensor() = 0;
+
+ private:
+  VRDeviceProvider* provider_;
+  unsigned int id_;
+
+  static unsigned int next_id_;
+};

[mdempsky, 2015/05/11 19:25:01]

Nit: DISALLOW_COPY_AND_ASSIGN.

[bajones, 2015/05/12 00:13:39]

Can't DISALLOW_COPY_AND_ASSIGN on an abstract class.

[mdempsky_google, 2015/05/12 00:25:15]

Can you elaborate?  Abstract classes still have implicit copy constructors and
assignment operators, and there are other abstract classes in Chromium that have
DISALLOW_COPY_AND_ASSIGN.  It's not obvious to me why it wouldn't work here too.

+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_VR_VR_DEVICE_H