Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(307)

Side by Side Diff: LayoutTests/inspector/profiler/heap-snapshot-test.js

Issue 218393010: DevTools: Fix PostOrder calculation algorithm in heap snapshot. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Addressing comments Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 var initialize_HeapSnapshotTest = function() { 1 var initialize_HeapSnapshotTest = function() {
2 2
3 InspectorTest.createHeapSnapshotMockFactories = function() { 3 InspectorTest.createHeapSnapshotMockFactories = function() {
4 4
5 InspectorTest.createJSHeapSnapshotMockObject = function() 5 InspectorTest.createJSHeapSnapshotMockObject = function()
6 { 6 {
7 return { 7 return {
8 _rootNodeIndex: 0, 8 _rootNodeIndex: 0,
9 _nodeTypeOffset: 0, 9 _nodeTypeOffset: 0,
10 _nodeNameOffset: 1, 10 _nodeNameOffset: 1,
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 }; 53 };
54 54
55 InspectorTest.createHeapSnapshotMockRaw = function() 55 InspectorTest.createHeapSnapshotMockRaw = function()
56 { 56 {
57 // Effectively the same graph as in createJSHeapSnapshotMockObject, 57 // Effectively the same graph as in createJSHeapSnapshotMockObject,
58 // but having full set of fields. 58 // but having full set of fields.
59 // 59 //
60 // A triple in parentheses indicates node index, self size and 60 // A triple in parentheses indicates node index, self size and
61 // retained size. 61 // retained size.
62 // 62 //
63 // --- A (14,2,2) --ac- C (40,4,10) -ce- E(57,6,6) 63 // --- A (7,2,2) --ac- C (21,4,10) -ce- E(35,6,6)
64 // / | / 64 // / | /
65 // "" (1,0,20) 1 --bc----- 65 // "" (0,0,20) 1 --bc-----
66 // \ v / 66 // \ v /
67 // --- B (27,3,8) --bd- D (50,5,5) 67 // --- B (14,3,8) --bd- D (28,5,5)
68 // 68 //
69 return { 69 return {
70 snapshot: { 70 snapshot: {
71 meta: { 71 meta: {
72 node_fields: ["type", "name", "id", "self_size", "retained_size" , "dominator", "edge_count"], 72 node_fields: ["type", "name", "id", "self_size", "retained_size" , "dominator", "edge_count"],
73 node_types: [["hidden", "object"], "", "", "", "", "", ""], 73 node_types: [["hidden", "object"], "", "", "", "", "", ""],
74 edge_fields: ["type", "name_or_index", "to_node"], 74 edge_fields: ["type", "name_or_index", "to_node"],
75 edge_types: [["element", "property", "shortcut"], "", ""] 75 edge_types: [["element", "property", "shortcut"], "", ""]
76 }, 76 },
77 node_count: 6, 77 node_count: 6,
(...skipping 648 matching lines...) Expand 10 before | Expand all | Expand 10 after
726 function sortingComplete() 726 function sortingComplete()
727 { 727 {
728 dataGrid.removeEventListener(WebInspector.HeapSnapshotSortableDataGr id.Events.SortingComplete, sortingComplete, null); 728 dataGrid.removeEventListener(WebInspector.HeapSnapshotSortableDataGr id.Events.SortingComplete, sortingComplete, null);
729 callback(); 729 callback();
730 } 730 }
731 dataGrid.addEventListener(WebInspector.HeapSnapshotSortableDataGrid.Even ts.SortingComplete, sortingComplete, null); 731 dataGrid.addEventListener(WebInspector.HeapSnapshotSortableDataGrid.Even ts.SortingComplete, sortingComplete, null);
732 } 732 }
733 }; 733 };
734 734
735 }; 735 };
OLDNEW
« no previous file with comments | « LayoutTests/inspector/profiler/heap-snapshot-expected.txt ('k') | Source/devtools/front_end/HeapSnapshot.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698