Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(9)

Issue 2899453002: Pass parsed network quality estimator params when constructing NQE

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 days, 12 hours ago by tbansal1
Modified:
12 hours, 17 minutes ago
CC:
chromium-reviews, cbentzel+watch_chromium.org, tbansal+watch-nqe_chromium.org, net-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Pass parsed network quality estimator params when constructing NQE This CL changes the way how Network Quality Estimator (NQE) is constructed. Currently, NQE configuration parameters are passed as key-value pair by Chrome/Cronet in the NQE's constructor. This CL requires Chrome/Cronet to parse the config params themselves, store them in a class, and pass an instance of that class in the constructor of NQE. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester BUG=638308

Patch Set 1 : ps #

Patch Set 2 : Fix Android compile error #

Total comments: 1

Patch Set 3 : Rebased #

Patch Set 4 : mmenke comments, rebased #

Patch Set 5 : fix Android compile error #

Unified diffs Side-by-side diffs Delta from patch set Stats (+201 lines, -206 lines) Patch
M chrome/browser/android/net/external_estimate_provider_android_unittest.cc View 1 2 3 4 2 chunks +7 lines, -3 lines 0 comments Download
M chrome/browser/io_thread.cc View 1 2 3 1 chunk +3 lines, -1 line 0 comments Download
M components/cronet/android/cronet_url_request_context_adapter.cc View 1 2 3 2 chunks +4 lines, -2 lines 0 comments Download
M ios/chrome/browser/ios_chrome_io_thread.mm View 1 2 3 2 chunks +4 lines, -1 line 0 comments Download
M net/nqe/network_quality_estimator.h View 1 2 3 3 chunks +15 lines, -28 lines 0 comments Download
M net/nqe/network_quality_estimator.cc View 1 2 3 18 chunks +48 lines, -59 lines 0 comments Download
M net/nqe/network_quality_estimator_params.h View 1 2 3 4 chunks +8 lines, -15 lines 0 comments Download
M net/nqe/network_quality_estimator_params.cc View 1 2 3 10 chunks +92 lines, -78 lines 0 comments Download
M net/nqe/network_quality_estimator_test_util.cc View 1 2 3 2 chunks +9 lines, -6 lines 0 comments Download
M net/nqe/network_quality_estimator_unittest.cc View 2 chunks +5 lines, -5 lines 0 comments Download
M net/nqe/throughput_analyzer.h View 1 2 3 1 chunk +1 line, -2 lines 0 comments Download
M net/nqe/throughput_analyzer_unittest.cc View 1 2 3 5 chunks +5 lines, -6 lines 0 comments Download
Commit queue not available (can’t edit this change).

Messages

Total messages: 51 (41 generated)
tbansal1
mmenke: ptal at io_thread*, and let me know if this is the right way for ...
6 days, 11 hours ago (2017-05-19 19:59:56 UTC) #22
tbansal1
mmenke: ping. Thanks.
2 days, 12 hours ago (2017-05-23 18:37:59 UTC) #29
mmenke
On 2017/05/23 18:37:59, tbansal1 wrote: > mmenke: ping. Thanks. I'm sorry for the delay - ...
2 days, 10 hours ago (2017-05-23 20:49:56 UTC) #30
mmenke
On 2017/05/23 18:37:59, tbansal1 wrote: > mmenke: ping. Thanks. I'm sorry for the delay - ...
2 days, 10 hours ago (2017-05-23 20:49:58 UTC) #31
tbansal1
On 2017/05/23 20:49:58, mmenke wrote: > On 2017/05/23 18:37:59, tbansal1 wrote: > > mmenke: ping. ...
2 days, 10 hours ago (2017-05-23 20:51:39 UTC) #32
mmenke
Sorry again for the delay - 3 days is just way too slow. Had also ...
1 day, 15 hours ago (2017-05-24 15:49:04 UTC) #33
tbansal1
mmenke: ptal. xunjieli: components/cronet/android/cronet_url_request_context_adapter.cc Thanks.
14 hours, 15 minutes ago (2017-05-25 17:00:58 UTC) #41
mmenke
On 2017/05/25 17:00:58, tbansal1 wrote: > mmenke: ptal. > > xunjieli: components/cronet/android/cronet_url_request_context_adapter.cc > > Thanks. ...
14 hours, 1 minute ago (2017-05-25 17:14:09 UTC) #42
tbansal1
rohitrao: ptal at ios/chrome/browser/ios_chrome_io_thread.mm ryansturm: //net/nqe. Thanks.
13 hours, 47 minutes ago (2017-05-25 17:28:58 UTC) #48
xunjieli
13 hours, 19 minutes ago (2017-05-25 17:56:42 UTC) #49
On 2017/05/25 17:00:58, tbansal1 wrote:
> mmenke: ptal. 
> 
> xunjieli: components/cronet/android/cronet_url_request_context_adapter.cc
> 
> Thanks.

cronet lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 650457f06