Bug 50100 - Web Inspector: UI polish for Network and Resources panels.
Summary: Web Inspector: UI polish for Network and Resources panels.
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-25 23:15 PST by Pavel Feldman
Modified: 2010-11-26 14:36 PST (History)
14 users (show)

See Also:

Attachments
[PATCH] Proposed change (13.25 KB, patch)
2010-11-25 23:18 PST, Pavel Feldman 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Pavel Feldman 2010-11-25 23:15:41 PST 
Patch to follow.
Comment 1 Pavel Feldman 2010-11-25 23:18:59 PST 
Created attachment 74905 [details]
[PATCH] Proposed change
Comment 2 Yury Semikhatsky 2010-11-26 04:44:01 PST 
Comment on attachment 74905 [details]
[PATCH] Proposed change

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

> WebCore/ChangeLog:5
> +        Web Inspector: UI polish for Network and Resources panels.

Could you add at least short descriptions for the changes made?

> WebCore/inspector/front-end/Popover.js:-195
> -        const toolTipDelay = isMouseDown ? 0 : (this._popup ? 600 : 1000);

1.2 -> 0.6
Comment 3 Pavel Feldman 2010-11-26 04:51:43 PST 
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/NetworkPanel.js
	M	WebCore/inspector/front-end/Popover.js
	M	WebCore/inspector/front-end/Resource.js
	M	WebCore/inspector/front-end/ResourceManager.js
	M	WebCore/inspector/front-end/ResourcesPanel.js
	M	WebCore/inspector/front-end/networkPanel.css
Committed r72758
Comment 4 Csaba Osztrogonác 2010-11-26 06:27:15 PST 
It broke many tests on all platform. Could you fix it?
Comment 5 Csaba Osztrogonác 2010-11-26 06:27:40 PST 
Comment on attachment 74905 [details]
[PATCH] Proposed change

remove r+ from landed patch
Comment 6 WebKit Review Bot 2010-11-26 07:38:49 PST 
http://trac.webkit.org/changeset/72758 might have broken GTK Linux 32-bit Debug
The following tests are not passing:
inspector/debugger-pause-on-debugger-statement.html
Comment 7 Csaba Osztrogonác 2010-11-26 14:36:40 PST 
fix landed in http://trac.webkit.org/changeset/72759