Bug 53828

Summary:

REGRESSION: Web Inspector says that all resources start download at once

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Major

CC:

beidson, koivisto, mihaip, pfeldman

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.6

Attachments:

Description	Flags
local test case	none
Safari 5.0.3 screenshot	none
nightly screenshot	none

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	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 attachment proposed patch, testcase, etc.

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

<rdar://problem/9054881>

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.