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+

Description Ilya Tikhonovsky 2012-02-29 07:29:24 PST
%subj%
Comment 1 Ilya Tikhonovsky 2012-02-29 07:31:49 PST
Created attachment 129449 [details]
Patch
Comment 2 Ilya Tikhonovsky 2012-02-29 08:53:20 PST
Created attachment 129459 [details]
Patch
Comment 3 Ilya Tikhonovsky 2012-02-29 22:23:29 PST
Created attachment 129635 [details]
polished patch
Comment 4 Ilya Tikhonovsky 2012-03-01 02:03:33 PST
Created attachment 129668 [details]
Patch
Comment 5 Vsevolod Vlasov 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.
Comment 6 Ilya Tikhonovsky 2012-03-01 05:59:50 PST
Created attachment 129692 [details]
Patch
Comment 7 Ilya Tikhonovsky 2012-03-01 06:15:45 PST
Committed r109343: <http://trac.webkit.org/changeset/109343>
Comment 8 Ilya Tikhonovsky 2012-03-12 07:00:07 PDT
Committed r110423: <http://trac.webkit.org/changeset/110423>