Bug 79911

Summary: Web Inspector: move heap snapshot nodes data to external array.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, apavlov, bweinstein, cc-bugs, jamesr, joepeck, keishi, loislo, macpherson, menard, ojan, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 80212    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
polished patch
none
Patch
none
Patch vsevik: review+

Ilya Tikhonovsky
Reported 2012-02-29 07:29:24 PST
%subj%
Attachments
Patch (3.24 KB, patch)
2012-02-29 07:31 PST, Ilya Tikhonovsky
no flags
Patch (5.32 KB, patch)
2012-02-29 08:53 PST, Ilya Tikhonovsky
no flags
polished patch (4.87 KB, patch)
2012-02-29 22:23 PST, Ilya Tikhonovsky
no flags
Patch (5.93 KB, patch)
2012-03-01 02:03 PST, Ilya Tikhonovsky
no flags
Patch (7.93 KB, patch)
2012-03-01 05:59 PST, Ilya Tikhonovsky
vsevik: review+
Ilya Tikhonovsky
Comment 1 2012-02-29 07:31:49 PST
Ilya Tikhonovsky
Comment 2 2012-02-29 08:53:20 PST
Ilya Tikhonovsky
Comment 3 2012-02-29 22:23:29 PST
Created attachment 129635 [details] polished patch
Ilya Tikhonovsky
Comment 4 2012-03-01 02:03:33 PST
Vsevolod Vlasov
Comment 5 2012-03-01 05:00:28 PST
Comment on attachment 129668 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=129668&action=review > Source/WebCore/inspector/front-end/HeapSnapshot.js:739 > + else { // This is a fallback for heap-snapshot-loader.html Layout test. Could you please move the code in else to the test.
Ilya Tikhonovsky
Comment 6 2012-03-01 05:59:50 PST
Ilya Tikhonovsky
Comment 7 2012-03-01 06:15:45 PST
Ilya Tikhonovsky
Comment 8 2012-03-12 07:00:07 PDT
Note You need to log in before you can comment on or make changes to this bug.