Bug 107291

Summary: Web Inspector: data grid resize is slow
Product: WebKit Reporter: Andrey Kosyakov <caseq>
Component: Web Inspector (Deprecated)Assignee: Andrey Kosyakov <caseq>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch pfeldman: review+

Andrey Kosyakov
Reported 2013-01-18 09:05:00 PST
0. Open inspector 1. Switch to network panel (or Resources -> Cookies) 2. Quickly drag a resizer in the table 3. Observe update lags considerably
Attachments
Patch (3.03 KB, patch)
2013-01-18 09:07 PST, Andrey Kosyakov
pfeldman: review+
Andrey Kosyakov
Comment 1 2013-01-18 09:07:20 PST
Pavel Feldman
Comment 2 2013-01-20 23:22:42 PST
Comment on attachment 183475 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183475&action=review > Source/WebCore/inspector/front-end/DataGrid.js:688 > + resizer._position = left; Resizers are cached and I don't see where this property is reset. Do you want to split this loop into two loops? First would measure, second would apply.
Andrey Kosyakov
Comment 3 2013-01-21 00:54:29 PST
Comment on attachment 183475 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183475&action=review >> Source/WebCore/inspector/front-end/DataGrid.js:688 >> + resizer._position = left; > > Resizers are cached and I don't see where this property is reset. Do you want to split this loop into two loops? First would measure, second would apply. Why would we have to reset this property? It just mirrors resize.style.left, so its life time is meant to match the life time of resizer.
Andrey Kosyakov
Comment 4 2013-01-21 08:09:23 PST
Note You need to log in before you can comment on or make changes to this bug.