Bug 53397 - Web Inspector: speed up network panel rendering.
Summary: Web Inspector: speed up network panel rendering.
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: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-30 08:49 PST by Pavel Feldman
Modified: 2011-01-31 06:45 PST (History)
10 users (show)

See Also:


Attachments
Patch (4.84 KB, patch)
2011-01-30 08:52 PST, Pavel Feldman
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2011-01-30 08:49:45 PST
Network panel is recalculating styles for ages. Open standalone front-end and execute

for (var i = 0; i < 1000; ++i) WebInspector.networkManager._dispatcher.didLoadResourceFromMemoryCache(new Date().getTime(), {url:"foo", response: {}, loader: { url: "foo" }});

in the console. It'll fill network panel with 1K entries. Now try switching to/from panel. Also try showing resource content and closing it. You will see 600ms delay upon panel switch and 2s delay upon resource content close. We need to split content using iframes, but this is a long road.

This patch will improve things a lot in the short term.
Comment 1 Pavel Feldman 2011-01-30 08:52:31 PST
Created attachment 80591 [details]
Patch
Comment 2 Timothy Hatcher 2011-01-30 10:47:24 PST
Comment on attachment 80591 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=80591&action=review

> Source/WebCore/inspector/front-end/NetworkPanel.js:1032
> +    _onScroll: function(e)

I prefer "_didScroll" or "_updateOffscreenRows" over the generic "onScroll".
Comment 3 Pavel Feldman 2011-01-31 06:45:15 PST
Committed r77127: <http://trac.webkit.org/changeset/77127>