New issue
Advanced search Search tips

Issue 700215 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Expose RTT and throughput estimates on UI thread

Project Member Reported by tbansal@chromium.org, Mar 10 2017

Issue description

NQE should expose RTT and throughput estimates on UI thread.

The immediate use case is to eventually plumb them to the renderer process, and then to the Blink.

In long term, this is also necessary for exposing the estimates via JS API and/or HTTP headers.
 
Components: Internals>Network>NetworkQuality
Labels: OS-All
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bddab11adb5f2abcf364ba53f8382df1d0b3098d

commit bddab11adb5f2abcf364ba53f8382df1d0b3098d
Author: tbansal <tbansal@chromium.org>
Date: Sat Mar 11 01:50:46 2017

NQE: Plumb RTT and throughput estimates to the UI thread

This CL makes the RTT and throughput estimates available on the UI
thread. It also makes it possible to add observers on UI thread that
can listen to changes in the RTT/throughput estimations.

BUG= 700215 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bmcquade@chromium.org, petewil@chromium.org

Review-Url: https://codereview.chromium.org/2717153002
Cr-Commit-Position: refs/heads/master@{#456252}

[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/net/nqe/ui_network_quality_estimator_service.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/net/nqe/ui_network_quality_estimator_service.h
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/net/nqe/ui_network_quality_estimator_service_test_util.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/net/nqe/ui_network_quality_estimator_service_test_util.h
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/chrome/browser/page_load_metrics/observers/ukm_page_load_metrics_observer_unittest.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/components/offline_pages/core/background/network_quality_provider_stub.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/components/offline_pages/core/background/network_quality_provider_stub.h
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/net/nqe/network_quality_estimator.cc
[modify] https://crrev.com/bddab11adb5f2abcf364ba53f8382df1d0b3098d/net/nqe/network_quality_estimator.h

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 16 2017

Labels: M-59
Status: Fixed (was: Started)

Sign in to add a comment