Makoto Kato [:m_kato] Assignee
	Description • 12 years ago

I have the PC that has ambient light sensor...  We can get it using Sensor API (from Windows 7 or later).

	Doug Turner (:dougt)
	Comment 2 • 12 years ago

Comment on attachment 623602 [details] [diff] [review]
fix

Review of attachment 623602 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine.  Let me know about the filter.  Fix the other two nits if you like.

::: hal/windows/WindowsSensor.cpp
@@ +118,5 @@
> +    HRESULT hr;
> +    InfallibleTArray<float> values;
> +
> +    // Illuminance level, in lux
> +    hr = aReport->GetSensorValue(SENSOR_DATA_TYPE_LIGHT_LEVEL_LUX, &v);

do you know why they are running a filter over the sensor data in this example:

  http://msdn.microsoft.com/en-us/library/windows/desktop/dd318933%28v=vs.85%29.aspx

@@ +140,5 @@
>  {
> +  // Currently implementation is acceleration and ambient light only.
> +  if ((aSensor == SENSOR_ACCELERATION && sAccelerometer) ||
> +      (aSensor == SENSOR_LIGHT && sLight) ||
> +      (aSensor != SENSOR_ACCELERATION && aSensor != SENSOR_LIGHT)) {

This if statement is kind of hard to follow.  I would have two statements - one that just tests if we aren't SENSOR_ACCELERATION or SENSOR_LIGHT.  Have that return early.  

if (aSensor != SENSOR_LIGHT || aSensor != SENSOR_ACCELERATION) {
  return;
}

if ((aSensor == SENSOR_ACCELERATION && !sAccelerometer) ||
    (aSensor == SENSOR_LIGHT && !sLight)  {
...


Up to you.

@@ +194,5 @@
> +                                   getter_AddRefs(values)))) {
> +      if (SUCCEEDED(values->SetUnsignedIntegerValue(
> +                      SENSOR_PROPERTY_CURRENT_REPORT_INTERVAL,
> +                      DEFAULT_SENSOR_POLL))) {
> +        nsRefPtr<IPortableDeviceValues> returns;

While we are here, lets rename returns to result.

	Reuben Morais [:reuben]
	Comment 3 • 11 years ago

What happened to this patch? Are you still working on it, Makoto?

	Makoto Kato [:m_kato] Assignee
	Comment 4 • 11 years ago

ah, I forget this bug.  I will rebase.

	Stefan Weiss [:sir_none]
	Comment 5 • 10 years ago

Hi Makoto,
again one year over -> patch will come in next time?

	Makoto Kato [:m_kato] Assignee
	Comment 6 • 10 years ago

(In reply to Stefan Weiss [:sir_none] from comment #5)
> Hi Makoto,
> again one year over -> patch will come in next time?

I will work this next week

	Makoto Kato [:m_kato] Assignee
	Comment 7 • 6 years ago

This feature will be turned off, so I mark as wontfix.