NetworkQualityEstimator: Update signal strength estimate periodically

NetworkQualityEstimator: Update signal strength estimate periodically

Currently, the signal strength is updated when a main frame request
is received. That approach does not work for net embedders that do
not have a main frame concept (e.g., via cronet). This CL changes NQE
to update the signal strength periodically without relying on the main frame requests.

BUG=681929
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2650573002
Cr-Commit-Position: refs/heads/master@{#446329}
Committed: https://chromium.googlesource.com/chromium/src/+/190e126297503e8c5da6bd72390af82dc757a897

[tbansal1, 2017-01-21 09:52:44]

Description was changed from

==========
NQE signal; stremgth

BUG=
==========

to

==========
NQE signal; strength

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-01-21 09:53:13]

The CQ bit was checked by tbansal@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-21 09:53:24]

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, 2017-01-21 11:54:13]

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

[commit-bot: I haz the power, 2017-01-21 11:54:14]

Dry run: This issue passed the CQ dry run.

[tbansal1, 2017-01-23 20:47:24]

Description was changed from

==========
NQE signal; strength

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
NetworkQualityEstimator: Update signal strength estimate periodically

Currently, the signal strength is updated when a main frame request
is received. That approach does not work for net embedders that do
not have a main frame concept (e.g., via cronet). This CL changes NQE
to update the signal strength periodically without relying on the main frame
requests.

BUG=681929
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-01-23 20:47:39]

tbansal@chromium.org changed reviewers:
+ bengr@chromium.org

[tbansal1, 2017-01-23 20:47:39]

bengr: ptal. Thanks.

[tbansal1, 2017-01-25 18:03:17]

tbansal@chromium.org changed reviewers:
+ ryansturm@chromium.org
- bengr@chromium.org

[tbansal1, 2017-01-25 18:03:18]

ryansturm: ptal. Thanks.

[RyanSturm, 2017-01-25 21:12:02]

Thanks for fixing this. lgtm

[tbansal1, 2017-01-25 21:13:20]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-01-25 21:14:01]

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, 2017-01-26 00:23:40]

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

[commit-bot: I haz the power, 2017-01-26 00:23:41]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[tbansal1, 2017-01-26 00:25:16]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-01-26 00:26:01]

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, 2017-01-26 03:49:58]

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

[commit-bot: I haz the power, 2017-01-26 03:49:59]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[tbansal1, 2017-01-26 04:20:31]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-01-26 04:21:11]

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, 2017-01-26 07:25:21]

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

[commit-bot: I haz the power, 2017-01-26 07:25:22]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[tbansal1, 2017-01-26 13:49:00]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-01-26 13:49:16]

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, 2017-01-26 15:07:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485438540069940, "parent_rev":
"1498add8ed79bc971a9f786662de9716270fad7c", "commit_rev":
"190e126297503e8c5da6bd72390af82dc757a897"}

[commit-bot: I haz the power, 2017-01-26 15:07:42]

Description was changed from

==========
NetworkQualityEstimator: Update signal strength estimate periodically

Currently, the signal strength is updated when a main frame request
is received. That approach does not work for net embedders that do
not have a main frame concept (e.g., via cronet). This CL changes NQE
to update the signal strength periodically without relying on the main frame
requests.

BUG=681929
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
NetworkQualityEstimator: Update signal strength estimate periodically

Currently, the signal strength is updated when a main frame request
is received. That approach does not work for net embedders that do
not have a main frame concept (e.g., via cronet). This CL changes NQE
to update the signal strength periodically without relying on the main frame
requests.

BUG=681929
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2650573002
Cr-Commit-Position: refs/heads/master@{#446329}
Committed:
https://chromium.googlesource.com/chromium/src/+/190e126297503e8c5da6bd72390a...
==========

[commit-bot: I haz the power, 2017-01-26 15:07:44]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/190e126297503e8c5da6bd72390a...