Bug 53828 - REGRESSION: Web Inspector says that all resources start download at once
Summary: REGRESSION: Web Inspector says that all resources start download at once
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.6
: P1 Major
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2011-02-04 17:45 PST by Alexey Proskuryakov
Modified: 2011-08-08 22:50 PDT (History)
4 users (show)

See Also:

Attachments
local test case (1.16 KB, text/html)
2011-02-04 17:45 PST, Alexey Proskuryakov 		no flags Details
Safari 5.0.3 screenshot (189.37 KB, image/png)
2011-02-04 17:46 PST, Alexey Proskuryakov 		no flags Details
nightly screenshot (276.06 KB, image/png)
2011-02-04 17:47 PST, Alexey Proskuryakov 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 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.
Comment 1 Alexey Proskuryakov 2011-02-04 17:46:10 PST 
Created attachment 81329 [details]
Safari 5.0.3 screenshot
Comment 2 Alexey Proskuryakov 2011-02-04 17:47:54 PST 
Created attachment 81330 [details]
nightly screenshot
Comment 3 Alexey Proskuryakov 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.
Comment 4 Alexey Proskuryakov 2011-02-06 16:44:54 PST 
Looks like a Web inspector issue, not a loader one.
Comment 5 Alexey Proskuryakov 2011-02-25 11:18:08 PST 
<rdar://problem/9054881>
Comment 6 Pavel Feldman 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?
Comment 7 Pavel Feldman 2011-08-08 22:50:25 PDT 
Can't repro