Remove first check random delay on older devices

Remove first check random delay on older devices

BUG=598841
TEST=Not easily testable, as this CL removes code that is no longer
active after a first Quirks server check; we expect nearly all devices
should have made a server check as of M-54, if they're ever going to.

Committed: https://crrev.com/0bbe13103a5a5ce3940a529d986f68ded2e19ac6
Cr-Commit-Position: refs/heads/master@{#404587}

[Greg Levin, 2016-07-08 23:18:41]

glevin@chromium.org changed reviewers:
+ oshima@chromium.org

[Greg Levin, 2016-07-08 23:18:42]

Removing random 0-30 delay for first server contact.  This was to spread out
initial server traffic.  Now that QC has been live for a few releases, this is
no longer necessary.

Please have a look!

https://codereview.chromium.org/2006143011/diff/1/components/quirks/quirks_ma...
File components/quirks/quirks_manager.cc (right):

https://codereview.chromium.org/2006143011/diff/1/components/quirks/quirks_ma...
components/quirks/quirks_manager.cc:50: // TODO(glevin): Awaiting final decision
on how to handle old read-only files.
TODO: Remove this TODO

[oshima, 2016-07-09 00:31:17]

lgtm

[Greg Levin, 2016-07-10 15:30:10]

https://codereview.chromium.org/2006143011/diff/1/components/quirks/quirks_ma...
File components/quirks/quirks_manager.cc (right):

https://codereview.chromium.org/2006143011/diff/1/components/quirks/quirks_ma...
components/quirks/quirks_manager.cc:50: // TODO(glevin): Awaiting final decision
on how to handle old read-only files.
On 2016/07/08 23:18:42, Greg Levin wrote:
> TODO: Remove this TODO

Done.

[Greg Levin, 2016-07-10 15:30:49]

The CQ bit was checked by glevin@chromium.org

[Greg Levin, 2016-07-10 15:30:49]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2006143011/#ps20001
(title: "Remove obsolete TODO")

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

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-07-11 01:15:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-11 01:19:26]

Description was changed from

==========
Remove first check random delay on older devices

BUG=598841
TEST=Not easily testable, as this CL removes code that is no longer
active after a first Quirks server check; we expect nearly all devices
should have made a server check as of M-54, if they're ever going to.
==========

to

==========
Remove first check random delay on older devices

BUG=598841
TEST=Not easily testable, as this CL removes code that is no longer
active after a first Quirks server check; we expect nearly all devices
should have made a server check as of M-54, if they're ever going to.

Committed: https://crrev.com/0bbe13103a5a5ce3940a529d986f68ded2e19ac6
Cr-Commit-Position: refs/heads/master@{#404587}
==========

[commit-bot: I haz the power, 2016-07-11 01:19:27]

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