Chromium Code Reviews
DescriptionRevert of NQE: Record the main frame metrics at transaction start (patchset #2 id:60001 of https://codereview.chromium.org/2695783003/ )
Reason for revert:
Caused perf regression. See https://bugs.chromium.org/p/chromium/issues/detail?id=692933
Original issue's description:
> NQE: Record the main frame metrics at transaction start
>
> In network quality estimator (NQE), record the main frame metrics at the
> beginning of transaction start, instead of when the response headers have
> been received for the main frame.
>
> BUG=691798
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
>
> Review-Url: https://codereview.chromium.org/2695783003
> Cr-Commit-Position: refs/heads/master@{#450536}
> Committed: https://chromium.googlesource.com/chromium/src/+/0be260cab40bb7162d01d99afb1144abdd2469c2
TBR=ryansturm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=691798, 692933
Review-Url: https://codereview.chromium.org/2690303011
Cr-Commit-Position: refs/heads/master@{#451106}
Committed: https://chromium.googlesource.com/chromium/src/+/c82d9e4024d4ada2d0a0677c2ced1f1147dddb7c
Patch Set 1 #
Messages
Total messages: 7 (4 generated)
|
|||||||||||||||||||||||||||||||||||||