Implement Navigation.TimeToNetworkStack UMA |
||
Issue descriptionI've noticed a trend where navigation throttles observe WillStartRequest, and post a task to the IO thread to do some async work. This IO thread work blocks navigation!!! I.e. since the IO thread is FIFO this work will occur before NavigationURLLoaderImplCore::Start. In fact, I think issue 779914 might be in this critical path. I firmly believe adding this metric will prevent PLT regressions moving forward, as it will be extremely low noise, and 100% in the critical path of loading (if restricted to main frames).
,
Nov 2 2017
Oh wait, is this Navigation.TimeToURLJobStart?
,
Nov 2 2017
yupp, thanks carlosk for doing this for me :D
,
Nov 7 2017
You're welcome! But please note that there is the intention to remove this and other Navigation.TimeTo* histograms once PlzNavigate is fully landed. See issue 698957. |
||
►
Sign in to add a comment |
||
Comment 1 by csharrison@chromium.org
, Nov 2 2017