[sensors] PlatformSensorProvider implementations are notified when there are no sensor instances

[sensors] PlatformSensorProvider implementations are notified when there are no sensor instances

The protected virtual 'AllSensorsRemoved' method is added to PlatformSensorProviderBase class.
Unit tests are also included.

BUG=606766
Committed: https://crrev.com/90f1d34ecc7699705c95df05e1b04f921007dba6
Cr-Commit-Position: refs/heads/master@{#427049}

[Mikhail, 2016-10-19 18:59:46]

The CQ bit was checked by mikhail.pozdnyakov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-19 19:00:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mikhail, 2016-10-19 19:00:39]

Description was changed from

==========
[sensors] PlatformSensorProvider implementations are notified when there are no
sensor instances

The protected virtual 'AllSensorsRemoved' method is added to
PlatformSensorProviderBase class.
Unit tests are also included.

BUG=606766
==========

to

==========
[sensors] PlatformSensorProvider implementations are notified when there are no
sensor instances

The protected virtual 'AllSensorsRemoved' method is added to
PlatformSensorProviderBase class.
Unit tests are also included.

BUG=606766
==========

[Mikhail, 2016-10-19 19:00:40]

mikhail.pozdnyakov@intel.com changed reviewers:
+ alexander.shalamov@intel.com, timvolodine@chromium.org

[Mikhail, 2016-10-19 19:46:54]

PTAL

[commit-bot: I haz the power, 2016-10-19 19:53:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 19:53:08]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Mikhail, 2016-10-19 20:01:49]

The CQ bit was checked by mikhail.pozdnyakov@intel.com to run a CQ dry run

[Mikhail, 2016-10-19 20:02:42]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-19 20:03:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 22:06:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 22:06:18]

Dry run: This issue passed the CQ dry run.

[shalamov, 2016-10-20 06:37:26]

https://codereview.chromium.org/2434073002/diff/20001/device/generic_sensor/p...
File device/generic_sensor/platform_sensor_provider_base.h (right):

https://codereview.chromium.org/2434073002/diff/20001/device/generic_sensor/p...
device/generic_sensor/platform_sensor_provider_base.h:43: virtual void
AllSensorsRemoved() {}
Can you also add bool HasSensors() const;

[Mikhail, 2016-10-20 08:03:40]

The CQ bit was checked by mikhail.pozdnyakov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-20 08:03:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mikhail, 2016-10-20 08:04:19]

On 2016/10/20 06:37:26, shalamov wrote:
>
https://codereview.chromium.org/2434073002/diff/20001/device/generic_sensor/p...
> File device/generic_sensor/platform_sensor_provider_base.h (right):
> 
>
https://codereview.chromium.org/2434073002/diff/20001/device/generic_sensor/p...
> device/generic_sensor/platform_sensor_provider_base.h:43: virtual void
> AllSensorsRemoved() {}
> Can you also add bool HasSensors() const;

Done.

[shalamov, 2016-10-20 08:08:11]

lgtm

[commit-bot: I haz the power, 2016-10-20 09:22:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-20 09:22:24]

Dry run: This issue passed the CQ dry run.

[Mikhail, 2016-10-24 06:26:38]

The CQ bit was checked by mikhail.pozdnyakov@intel.com

[commit-bot: I haz the power, 2016-10-24 06:26:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 06:28:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-24 06:28:43]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Mikhail, 2016-10-24 07:28:13]

The CQ bit was checked by mikhail.pozdnyakov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-24 07:28:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 09:44:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-24 09:44:20]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Mikhail, 2016-10-24 09:58:45]

The CQ bit was checked by mikhail.pozdnyakov@intel.com

[Mikhail, 2016-10-24 09:58:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
alexander.shalamov@intel.com
Link to the patchset: https://codereview.chromium.org/2434073002/#ps60001
(title: "rebased")

[commit-bot: I haz the power, 2016-10-24 09:58:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 12:02:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-24 12:02:50]

Failed to apply the patch.
On branch working_branch
Your branch is up-to-date with 'origin/refs/pending/heads/master'.
nothing to commit, working tree clean

[Mikhail, 2016-10-24 12:04:05]

The CQ bit was checked by mikhail.pozdnyakov@intel.com

[commit-bot: I haz the power, 2016-10-24 12:04:11]

Description was changed from

==========
[sensors] PlatformSensorProvider implementations are notified when there are no
sensor instances

The protected virtual 'AllSensorsRemoved' method is added to
PlatformSensorProviderBase class.
Unit tests are also included.

BUG=606766
==========

to

==========
[sensors] PlatformSensorProvider implementations are notified when there are no
sensor instances

The protected virtual 'AllSensorsRemoved' method is added to
PlatformSensorProviderBase class.
Unit tests are also included.

BUG=606766

Committed: https://crrev.com/90f1d34ecc7699705c95df05e1b04f921007dba6
Cr-Commit-Position: refs/heads/master@{#427049}
==========

[commit-bot: I haz the power, 2016-10-24 12:04:13]

Patchset 3 (id:??) landed as
https://crrev.com/90f1d34ecc7699705c95df05e1b04f921007dba6
Cr-Commit-Position: refs/heads/master@{#427049}