QuicStreamFactoryJob uses priority LOWEST for all its DNS requests |
|||
Issue descriptionhttps://cs.chromium.org/chromium/src/net/quic/chromium/quic_stream_factory.cc?type=cs&l=513 I'd add statistics on how much of a problem this is, but the UMA dashboard isn't loading. My guess is it's pretty minor.
,
Nov 13 2017
,
Nov 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a commit 247ea64167abfb6e96d0baa3fbbcb99a74f4c01a Author: Yixin Wang <wangyix@chromium.org> Date: Wed Nov 15 01:15:50 2017 Use the request priority in QuicStreamFactory::Job's host resolution step instead of DEFAULT_PRIORITY This adds a RequestPriority param to QuicStreamRequest::Request() Bug: 768043 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I9feb7f4d66eb971e6c66aa444317116c03dbdc2e Reviewed-on: https://chromium-review.googlesource.com/767853 Reviewed-by: Ryan Hamilton <rch@chromium.org> Commit-Queue: Yixin Wang <wangyix@chromium.org> Cr-Commit-Position: refs/heads/master@{#516542} [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/http/http_proxy_client_socket_wrapper.cc [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/http/http_stream_factory_impl_job.cc [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/quic/chromium/quic_stream_factory.cc [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/quic/chromium/quic_stream_factory.h [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/quic/chromium/quic_stream_factory_fuzzer.cc [modify] https://crrev.com/247ea64167abfb6e96d0baa3fbbcb99a74f4c01a/net/quic/chromium/quic_stream_factory_test.cc
,
Nov 17 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rch@chromium.org
, Nov 10 2017