[DIAL API] Force increment registry generation on discoverNow.

[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.

BUG=595204
Committed: https://crrev.com/7755e55177abe8fa6f42eaaabffaeeaa167a6161
Cr-Commit-Position: refs/heads/master@{#381823}

[imcheng, 2016-03-17 17:34:18]

imcheng@chromium.org changed reviewers:
+ mfoltz@chromium.org

[imcheng, 2016-03-17 17:34:19]

This can be something that we merge back to M50, but not required since no other
clients that I am aware of that rely on the incorrect assumption. Still I think
it would be nice to add a way to reliably get the current state of things for
the time being until we add the new API.

[imcheng, 2016-03-17 17:34:36]

Description was changed from

==========
[DIAL API] Force increment registry generation on discoverNow.

discoverNow is supposed to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.


BUG=595204
==========

to

==========
[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.


BUG=595204
==========

[imcheng, 2016-03-17 17:46:36]

Actually I have been informed that this will still be an issue for our client
running with the MR extension in M50. So to avoid doing a similar hack for the
client, we should land this fix.

[mark a. foltz, 2016-03-17 21:56:30]

lgtm

Please add a unit test for this (can be done in a separate patch if you want the
merge to be small).

[imcheng, 2016-03-17 21:59:58]

Yep, will be adding test separately.

[imcheng, 2016-03-17 22:05:23]

The CQ bit was checked by imcheng@chromium.org

[commit-bot: I haz the power, 2016-03-17 22:05:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810433002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810433002/20001

[commit-bot: I haz the power, 2016-03-17 22:59:27]

Description was changed from

==========
[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.


BUG=595204
==========

to

==========
[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.


BUG=595204
==========

[commit-bot: I haz the power, 2016-03-17 22:59:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-17 23:01:24]

Description was changed from

==========
[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.


BUG=595204
==========

to

==========
[DIAL API] Force increment registry generation on discoverNow.

Change discoverNow to fire the deviceList event with the latest
device list even if the list had not changed. The patch ensures that
by incrementing the registry generation if discovery was started
during discoverNow.

BUG=595204

Committed: https://crrev.com/7755e55177abe8fa6f42eaaabffaeeaa167a6161
Cr-Commit-Position: refs/heads/master@{#381823}
==========

[commit-bot: I haz the power, 2016-03-17 23:01:25]

Patchset 2 (id:??) landed as
https://crrev.com/7755e55177abe8fa6f42eaaabffaeeaa167a6161
Cr-Commit-Position: refs/heads/master@{#381823}