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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: LayoutTests/inspector/profiler/heap-snapshot-containment-expansion-preserved-when-sorting.html

Issue 211273005: Do not override data getter in heap snapshot grid nodes (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Removed unused parameter from findRow Created 6 years, 9 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
« no previous file with comments | « LayoutTests/inspector/profiler/heap-snapshot-comparison-shown-node-count-preserved-when-sorting.html ('k') | LayoutTests/inspector/profiler/heap-snapshot-containment-show-all.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script src="../../http/tests/inspector/inspector-test.js"></script> 3 <script src="../../http/tests/inspector/inspector-test.js"></script>
4 <script src="heap-snapshot-test.js"></script> 4 <script src="heap-snapshot-test.js"></script>
5 <script> 5 <script>
6 6
7 function test() 7 function test()
8 { 8 {
9 var instanceCount = 25; 9 var instanceCount = 25;
10 function createHeapSnapshot() 10 function createHeapSnapshot()
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 43
44 function step5(row) 44 function step5(row)
45 { 45 {
46 var row = row.children[0]; 46 var row = row.children[0];
47 InspectorTest.assertEquals(true, !!row, "\"B\" instance row"); 47 InspectorTest.assertEquals(true, !!row, "\"B\" instance row");
48 InspectorTest.expandRow(row, expandA); 48 InspectorTest.expandRow(row, expandA);
49 function expandA(row) 49 function expandA(row)
50 { 50 {
51 function propertyMatcher(data) 51 function propertyMatcher(data)
52 { 52 {
53 return data.name === "a" && data.value.charAt(0) === "A" ; 53 return data._referenceName === "a" && data._name.charAt( 0) === "A";
54 } 54 }
55 var aRow = InspectorTest.findRow("object", propertyMatcher, row); 55 var aRow = InspectorTest.findMatchingRow(propertyMatcher, ro w);
56 InspectorTest.assertEquals(true, !!aRow, "\"a: A\" row"); 56 InspectorTest.assertEquals(true, !!aRow, "\"a: A\" row");
57 InspectorTest.expandRow(aRow, step6); 57 InspectorTest.expandRow(aRow, step6);
58 } 58 }
59 } 59 }
60 60
61 var columnContents; 61 var columnContents;
62 function step6() 62 function step6()
63 { 63 {
64 columnContents = InspectorTest.columnContents(columns[0]); 64 columnContents = InspectorTest.columnContents(columns[0]);
65 InspectorTest.clickColumn(columns[0], clickTwice); 65 InspectorTest.clickColumn(columns[0], clickTwice);
(...skipping 15 matching lines...) Expand all
81 81
82 </script> 82 </script>
83 </head> 83 </head>
84 <body onload="runTest()"> 84 <body onload="runTest()">
85 <p> 85 <p>
86 Tests Containment view of detailed heap snapshots. 86 Tests Containment view of detailed heap snapshots.
87 Expanded nodes must be preserved after sorting. 87 Expanded nodes must be preserved after sorting.
88 </p> 88 </p>
89 </body> 89 </body>
90 </html> 90 </html>
OLDNEW
« no previous file with comments | « LayoutTests/inspector/profiler/heap-snapshot-comparison-shown-node-count-preserved-when-sorting.html ('k') | LayoutTests/inspector/profiler/heap-snapshot-containment-show-all.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698