Reland: Refactor DeviceMotionEventPump to use //device/generic_sensor instead of //device/sensors

third_party/WebKit/public/platform/modules/device_orientation/WebDeviceMotionData.h

 /*
- * Copyright (C) 2013 Google Inc. All rights reserved.
+ * Copyright (C) 2017 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
  * contributors may be used to endorse or promote products derived from
  * this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef DeviceMotionDispatcher_h
-#define DeviceMotionDispatcher_h
+#ifndef WebDeviceMotionData_h
+#define WebDeviceMotionData_h
 
-#include "core/frame/PlatformEventDispatcher.h"
-#include "platform/heap/Handle.h"
-#include "platform/wtf/RefPtr.h"
-#include "public/platform/modules/device_orientation/WebDeviceMotionListener.h"
-
-namespace device {
-class MotionData;

[timvolodine, 2017/05/23 12:33:40]

I think this was introduced previously to get rid of the redundant
WebDevice*Data classes in blink, not sure why we would bring them back?

[juncai, 2017/05/23 21:33:55]

Since the //device/sensors directory will be removed eventually, and the motion
and orientation event will use //device/generic_sensors as their backend, so the
device::MotionData is not used any more, and it can be removed in this CL. But
we choose to not remove any code from //device/sensors until the refactoring of
device motion and orientation is done, and this can make the purpose of each CL
more clear.

[timvolodine, 2017/05/24 15:30:16]

The WebDevice*Data classes were explicitly removed in favor of moving the PODs
to device/sensors in this CL https://codereview.chromium.org/2415083002. IIRC
this was in context of the servicification project, and there is a more detailed
explanation in description of that patch. So I am naturally concerned how this
going back will have an impact on things? is it better to keep these POD
definitions in the device namespace?

(also the diff is wrong as it compares with DeviceMotionDispatcher)

[juncai, 2017/05/26 02:38:53]

These POD definitions will eventually be removed when the
//content/renderer/device_sensors directory is removed and all its logic is
implemented on the blink side. This is part of the onion-soup effort which I
mentioned as the step 3 in the design doc. This onion-soup step will happen
after //device/sensors directory is removed.

The diff is confusing, but it is a new added file instead of a modified file.

-}
+#include <string.h>
 
 namespace blink {
 
-class DeviceMotionData;
+class WebDeviceMotionData {
+ public:
+  WebDeviceMotionData()
+      : has_acceleration_x(false),
+        has_acceleration_y(false),
+        has_acceleration_z(false),
+        has_acceleration_including_gravity_x(false),
+        has_acceleration_including_gravity_y(false),
+        has_acceleration_including_gravity_z(false),
+        has_rotation_rate_alpha(false),
+        has_rotation_rate_beta(false),
+        has_rotation_rate_gamma(false) {}
+  WebDeviceMotionData(const WebDeviceMotionData& other) = default;
+  ~WebDeviceMotionData() {}
 
-// This class listens to device motion data and notifies all registered
-// controllers.
-class DeviceMotionDispatcher final
-    : public GarbageCollectedFinalized<DeviceMotionDispatcher>,
-      public PlatformEventDispatcher,
-      public WebDeviceMotionListener {
-  USING_GARBAGE_COLLECTED_MIXIN(DeviceMotionDispatcher);
+  double acceleration_x;
+  double acceleration_y;
+  double acceleration_z;
 
- public:
-  static DeviceMotionDispatcher& Instance();
-  ~DeviceMotionDispatcher() override;
+  double acceleration_including_gravity_x;
+  double acceleration_including_gravity_y;
+  double acceleration_including_gravity_z;
 
-  // Note that the returned object is owned by this class.
-  // FIXME: make the return value const, see crbug.com/233174.
-  DeviceMotionData* LatestDeviceMotionData();
+  double rotation_rate_alpha;
+  double rotation_rate_beta;
+  double rotation_rate_gamma;
 
-  // Inherited from WebDeviceMotionListener.
-  void DidChangeDeviceMotion(const device::MotionData&) override;
+  double interval;
 
-  DECLARE_VIRTUAL_TRACE();
+  bool has_acceleration_x : 1;
+  bool has_acceleration_y : 1;
+  bool has_acceleration_z : 1;
 
- private:
-  DeviceMotionDispatcher();
+  bool has_acceleration_including_gravity_x : 1;
+  bool has_acceleration_including_gravity_y : 1;
+  bool has_acceleration_including_gravity_z : 1;
 
-  // Inherited from PlatformEventDispatcher.
-  void StartListening() override;
-  void StopListening() override;
-
-  Member<DeviceMotionData> last_device_motion_data_;
+  bool has_rotation_rate_alpha : 1;
+  bool has_rotation_rate_beta : 1;
+  bool has_rotation_rate_gamma : 1;
 };
 
 }  // namespace blink
 
-#endif  // DeviceMotionDispatcher_h
+#endif  // WebDeviceMotionData_h