RESOLVED FIXED 53828
REGRESSION: Web Inspector says that all resources start download at once
https://bugs.webkit.org/show_bug.cgi?id=53828
Summary REGRESSION: Web Inspector says that all resources start download at once
Alexey Proskuryakov
Reported 2011-02-04 17:45:08 PST
Created attachment 81328 [details] local test case I'm not sure if this is an Inspector regression or a real loader one. It looks like all resources on a page start loading at once, ignoring per-host connection limit. Steps to reproduce: put attached test case into LayoutTests/http/tests/, and open <http://127.0.0.1:8000/test.html>. To reproduce again, reload.
Attachments
local test case (1.16 KB, text/html)
2011-02-04 17:45 PST, Alexey Proskuryakov
no flags
Safari 5.0.3 screenshot (189.37 KB, image/png)
2011-02-04 17:46 PST, Alexey Proskuryakov
no flags
nightly screenshot (276.06 KB, image/png)
2011-02-04 17:47 PST, Alexey Proskuryakov
no flags
Alexey Proskuryakov
Comment 1 2011-02-04 17:46:10 PST
Created attachment 81329 [details] Safari 5.0.3 screenshot
Alexey Proskuryakov
Comment 2 2011-02-04 17:47:54 PST
Created attachment 81330 [details] nightly screenshot
Alexey Proskuryakov
Comment 3 2011-02-04 17:49:36 PST
The connection limit of Snow Leopard is 6, and you can see that in Safari/WebKit 5.0.3, there are always no more than 6 connections. In nightly builds, it's either that Web Inspector lies, or the limit is not respected.
Alexey Proskuryakov
Comment 4 2011-02-06 16:44:54 PST
Looks like a Web inspector issue, not a loader one.
Alexey Proskuryakov
Comment 5 2011-02-25 11:18:08 PST
Pavel Feldman
Comment 6 2011-02-26 11:13:15 PST
We use InspectorInstrumentation::willSendRequest signal as a "start" mark. It looks like notifier now calls it for > than 6 resources at once (http://codesearch.google.com/codesearch/p?hl=en#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/loader/ResourceLoadNotifier.cpp&q=InspectorInstrumentation::willSendRequest&exact_package=chromium&sa=N&cd=3&ct=rc&l=121). Any idea on why it might be so?
Pavel Feldman
Comment 7 2011-08-08 22:50:25 PDT
Can't repro
Note You need to log in before you can comment on or make changes to this bug.