[Sensors] Remove SensorReading interfaces

third_party/WebKit/LayoutTests/sensor/resources/generic-sensor-tests.js

 'use strict';
 
 // Run a set of tests for a given |sensorType|. |updateReading| is
 // a called by the test to provide the mock values for sensor. |verifyReading|
 // is called so that the value read in JavaScript are the values expected (the ones
 // sent by |updateReading|).
 function runGenericSensorTests(sensorType, updateReading, verifyReading) {
   test(() => assert_throws(
     new RangeError(),
     () => new sensorType({frequency: -60})),
@@ skipping 152 matching lines @@
   function checkOnChangeIsCalledAndReadingIsValid(sensor) {
     let sensorObject = new sensorType({frequency: 60});
     sensorObject.start();
     let testPromise = sensor.mockSensorProvider.getCreatedSensor()
         .then(mockSensor => {
           return mockSensor.setUpdateSensorReadingFunction(updateReading);
         })
         .then((mockSensor) => {
           return new Promise((resolve, reject) => {
             let wrapper = new CallbackWrapper(() => {
-              assert_true(verifyReading(sensorObject.reading));
+              assert_true(verifyReading(sensorObject));
               sensorObject.stop();
-              assert_equals(sensorObject.reading, null);
+              assert_true(verifyReading(sensorObject, true /*should be null*/));
               resolve(mockSensor);
             }, reject);
 
             sensorObject.onchange = wrapper.callback;
             sensorObject.onerror = reject;
           });
         })
         .then(mockSensor => { return mockSensor.removeConfigurationCalled(); });
 
       return testPromise;
@@ skipping 11 matching lines @@
   sensor_test(sensor => {
     let sensorObject = new sensorType;
     sensorObject.start();
     let testPromise = sensor.mockSensorProvider.getCreatedSensor()
         .then(mockSensor => {
           return mockSensor.setUpdateSensorReadingFunction(updateReading);
         })
         .then(mockSensor => {
           return new Promise((resolve, reject) => {
             let wrapper = new CallbackWrapper(() => {
-              assert_true(verifyReading(sensorObject.reading));
+              assert_true(verifyReading(sensorObject));
               resolve(mockSensor);
             }, reject);
 
             sensorObject.onchange = wrapper.callback;
             sensorObject.onerror = reject;
           });
         })
         .then(mockSensor => {
           testRunner.setPageVisibility('hidden');
           return mockSensor.suspendCalled();
@@ skipping 21 matching lines @@
 
     let sensor2 = new sensorType({frequency: 20});
     sensor2.start();
     let testPromise = sensor.mockSensorProvider.getCreatedSensor()
         .then(mockSensor => {
           return mockSensor.setUpdateSensorReadingFunction(updateReading);
         })
         .then((mockSensor) => {
           return new Promise((resolve, reject) => {
             let wrapper = new CallbackWrapper(() => {
-              // Reading value is correct.
-              assert_true(verifyReading(sensor1.reading));
+              // Reading values are correct for both sensors.
+              assert_true(verifyReading(sensor1));
+              assert_true(verifyReading(sensor2));
 
-              // Both sensors share the same reading instance.
-              let reading = sensor1.reading;
-              assert_equals(reading, sensor2.reading);
-
-              // After first sensor stops its reading is null, reading for second
-              // sensor sensor remains.
+              // After first sensor stops its reading values are null,
+              // reading values for the second sensor sensor remain.
               sensor1.stop();
-              assert_equals(sensor1.reading, null);
-              assert_true(verifyReading(sensor2.reading));
+              assert_true(verifyReading(sensor1, true /*should be null*/));
+              assert_true(verifyReading(sensor2));
 
               sensor2.stop();
-              assert_equals(sensor2.reading, null);
+              assert_true(verifyReading(sensor2, true /*should be null*/));
 
-              // Cached reading remains.
-              assert_true(verifyReading(reading));
               resolve(mockSensor);
             }, reject);
 
             sensor1.onchange = wrapper.callback;
             sensor1.onerror = reject;
             sensor2.onerror = reject;
           });
         })
         .then(mockSensor => { return mockSensor.removeConfigurationCalled(); });
 
@@ skipping 53 matching lines @@
 
   sensor_test(sensor => {
     return checkFrequencyHintWorks(sensor);
   }, 'Test that frequency hint works (onchange reporting).');
 
   sensor_test(sensor => {
     sensor.mockSensorProvider.setContinuousReportingMode();
     return checkFrequencyHintWorks(sensor);
   }, 'Test that frequency hint works (continuous reporting).');
 }