Bug 49596 - Web Inspector: init front-end UI upon DOMContentLoaded, not loaded event.
Summary: Web Inspector: init front-end UI upon DOMContentLoaded, not loaded event.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pavel Feldman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-16 08:18 PST by Pavel Feldman
Modified: 2010-11-21 07:37 PST (History)
14 users (show)

See Also:


Attachments
[PATCH] Proposed change. (1.27 KB, patch)
2010-11-16 08:24 PST, Pavel Feldman
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.