[DeviceLight] Browser+java part

content/browser/device_sensors/data_fetcher_shared_memory_default.cc

Index: content/browser/device_sensors/data_fetcher_shared_memory_default.cc
diff --git a/content/browser/device_sensors/data_fetcher_shared_memory_default.cc b/content/browser/device_sensors/data_fetcher_shared_memory_default.cc
index 457fd8d5b474eeebdeb985cd91ac114b5dfdc258..ee952885b66d891fdbcb3ff3283c9d599cc593ef 100644
--- a/content/browser/device_sensors/data_fetcher_shared_memory_default.cc
+++ b/content/browser/device_sensors/data_fetcher_shared_memory_default.cc
@@ -34,7 +34,7 @@ static bool SetOrientationBuffer(
 namespace content {
 
 DataFetcherSharedMemory::DataFetcherSharedMemory()
-    : motion_buffer_(NULL), orientation_buffer_(NULL) {
+    : motion_buffer_(NULL), orientation_buffer_(NULL), light_buffer_(NULL) {
 }
 
 DataFetcherSharedMemory::~DataFetcherSharedMemory() {
@@ -54,6 +54,8 @@ bool DataFetcherSharedMemory::Start(ConsumerType consumer_type, void* buffer) {
       UMA_HISTOGRAM_BOOLEAN("InertialSensor.OrientationDefaultAvailable",
           false);
       return SetOrientationBuffer(orientation_buffer_, true);
+    case CONSUMER_TYPE_LIGHT:
+      return true;

[timvolodine, 2014/09/04 17:16:57]

I assume this is for non-android platforms. Should the API return Inf in that
case? as it is now it looks like javascript will never get a callback.

[riju_, 2014/09/08 09:26:17]

Ok, putting inf in the light buffer.

     default:
       NOTREACHED();
   }
@@ -66,6 +68,8 @@ bool DataFetcherSharedMemory::Stop(ConsumerType consumer_type) {
       return SetMotionBuffer(motion_buffer_, false);
     case CONSUMER_TYPE_ORIENTATION:
       return SetOrientationBuffer(orientation_buffer_, false);
+    case CONSUMER_TYPE_LIGHT:
+      return light_buffer_;

[timvolodine, 2014/09/04 17:16:57]

so this will return false because light_buffer_==0?

[riju_, 2014/09/08 09:26:17]

Done.

     default:
       NOTREACHED();
   }