Make deviceorientationabsolute confrom with spec on Chromium OS

content/browser/device_sensors/data_fetcher_shared_memory_chromeos.cc

 // 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.
 
 #include "content/browser/device_sensors/data_fetcher_shared_memory.h"
 
 #include "content/browser/device_sensors/sensor_manager_chromeos.h"
 
 namespace content {
 
@@ skipping 10 matching lines @@
 
   switch (consumer_type) {
     case CONSUMER_TYPE_MOTION:
       sensor_manager_->StartFetchingDeviceMotionData(
           static_cast<DeviceMotionHardwareBuffer*>(buffer));
       return true;
     case CONSUMER_TYPE_ORIENTATION:
       sensor_manager_->StartFetchingDeviceOrientationData(
           static_cast<DeviceOrientationHardwareBuffer*>(buffer));
       return true;
-    case CONSUMER_TYPE_ORIENTATION_ABSOLUTE:
+    case CONSUMER_TYPE_ORIENTATION_ABSOLUTE: {
+      orientation_absolute_buffer_ =
+          static_cast<DeviceOrientationHardwareBuffer*>(buffer);
+      // Absolute device orientation not available on Chrome OS, let the
+      // implementation fire an all-null event to signal this to blink.
+      orientation_absolute_buffer_->seqlock.WriteBegin();
+      orientation_absolute_buffer_->data.absolute = true;
+      orientation_absolute_buffer_->data.allAvailableSensorsAreActive = true;
+      orientation_absolute_buffer_->seqlock.WriteEnd();
+      return false;
+    }
     case CONSUMER_TYPE_LIGHT:
       NOTIMPLEMENTED();
       return false;
   }
   NOTREACHED();
   return false;
 }
 
 bool DataFetcherSharedMemory::Stop(ConsumerType consumer_type) {
   switch (consumer_type) {
     case CONSUMER_TYPE_MOTION:
       return sensor_manager_->StopFetchingDeviceMotionData();
     case CONSUMER_TYPE_ORIENTATION:
       return sensor_manager_->StopFetchingDeviceOrientationData();
     case CONSUMER_TYPE_ORIENTATION_ABSOLUTE:
+      // Even though absolute device orientation is not available on Chrome OS,
+      // properly cleanup, and fire an all-null event to blink.

[timvolodine, 2016/05/09 18:20:41]

there is no "fire event" here, maybe just remove the whole comment altogether,
it seems self-explanatory.

[jonross, 2016/05/10 18:01:08]

Done.

+      if (orientation_absolute_buffer_) {
+        orientation_absolute_buffer_->seqlock.WriteBegin();
+        orientation_absolute_buffer_->data.allAvailableSensorsAreActive = false;
+        orientation_absolute_buffer_->seqlock.WriteEnd();
+        orientation_absolute_buffer_ = nullptr;
+      }
+      return true;
     case CONSUMER_TYPE_LIGHT:
       NOTIMPLEMENTED();
       return false;
   }
   NOTREACHED();
   return false;
 }
 
 }  // namespace content