Bug 49596

Summary: Web Inspector: init front-end UI upon DOMContentLoaded, not loaded event.
Product: WebKit Reporter: Pavel Feldman <pfeldman>
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, apavlov, bweinstein, eric, joepeck, keishi, loislo, ossy, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed change. none

Description Pavel Feldman 2010-11-16 08:18:50 PST
This speeds up front-end opening a bit (~50ms in concatenated mode)
Comment 1 Pavel Feldman 2010-11-16 08:24:04 PST
Created attachment 73996 [details]
[PATCH] Proposed change.
Comment 2 Pavel Feldman 2010-11-16 08:43:47 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/inspector.js
Committed r72102
Comment 3 WebKit Review Bot 2010-11-16 09:04:34 PST
http://trac.webkit.org/changeset/72102 might have broken Qt Linux Release
The following tests are not passing:
inspector/elements-panel-structure.html
inspector/extensions.html
Comment 4 Csaba Osztrogonác 2010-11-16 10:21:50 PST
(In reply to comment #3)
> http://trac.webkit.org/changeset/72102 might have broken Qt Linux Release
> The following tests are not passing:
> inspector/elements-panel-structure.html
> inspector/extensions.html

These tests are still broken on Qt bot:
http://build.webkit.org/results/Qt%20Linux%20Release/r72102%20%2823781%29/results.html

It isn't a build issue, Qt handles correctly if you touch a js file.
(So http://trac.webkit.org/changeset/72110 can't fix this fail.)
Comment 5 Pavel Feldman 2010-11-21 07:37:10 PST
It was landed and not rolled out. Qt needs fixing harness I guess.