Adding Chrome-side WebVR interface

content/browser/vr/vr_service.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_SERVICE_H
+#define CONTENT_BROWSER_VR_VR_SERVICE_H
+
+#include <map>
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/memory/singleton.h"
+#include "content/browser/vr/vr_device.h"
+#include "content/browser/vr/vr_device_provider.h"
+#include "content/common/content_export.h"
+
+namespace blink {
+class WebVRDevice;
+}
+
+namespace content {
+
+class CONTENT_EXPORT VRService {
+ public:
+  // Returns the VRService singleton.
+  static VRService* GetInstance();
+
+  void Initialize();
+  void Shutdown();

[no sievers, 2015/03/19 01:24:52]

It's unclear who is supposed to call these. Initialize() gets called internally
(so could be a private method), and Shutdown() does not get called.

+  void GetVRDevices(std::vector<blink::WebVRDevice>* webvr_devices);
+  VRDevice* GetDevice(unsigned int index);
+
+ private:
+  friend struct DefaultSingletonTraits<VRService>;
+  friend class VRServiceTest;
+
+  VRService();
+
+  // Constructor for testing. Takes ownership of the given provider.
+  VRService(VRDeviceProvider* provider);

[no sievers, 2015/03/19 01:24:52]

nit: explicit

+  virtual ~VRService();
+
+  void RegisterProvider(VRDeviceProvider* provider);
+
+  DISALLOW_COPY_AND_ASSIGN(VRService);

[no sievers, 2015/03/19 01:24:52]

nit: move this to the bottom

+
+  typedef std::vector<VRDeviceProvider*> ProviderList;
+  ProviderList providers_;
+
+  typedef std::map<unsigned int, VRDevice*> DeviceMap;
+  DeviceMap devices_;
+
+  bool vr_initialized_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_VR_VR_SERVICE_H