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

Unified Diff: third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html

Issue 2444223002: [Devtools] Cleanup DataGrid's typecast and identifier naming (Closed)
Patch Set: changes Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html
diff --git a/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html b/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html
index e9825c2242f16bb9c9754783bb9d24a57465573e..36910cf1cbb111ad23301aaba1af664fb799dc81 100644
--- a/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html
+++ b/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-sorting-fields.html
@@ -62,12 +62,12 @@ function test()
function step4(newColumnState)
{
columns[currentColumn] = newColumnState;
- var columnName = columns[currentColumn].identifier;
+ var columnName = columns[currentColumn].id;
var contents = windowRow.children.map(function(obj) { return obj.element().children[currentColumn].textContent; });
InspectorTest.assertEquals(instanceCount, contents.length, "column contents");
var sortTypes = { object: "text", distance: "number", count: "number", shallowSize: "size", retainedSize: "size" };
- InspectorTest.assertEquals(true, !!sortTypes[columns[currentColumn].identifier], "sort by identifier");
- InspectorTest.checkArrayIsSorted(contents, sortTypes[columns[currentColumn].identifier], columns[currentColumn].sort);
+ InspectorTest.assertEquals(true, !!sortTypes[columns[currentColumn].id], "sort by id");
+ InspectorTest.checkArrayIsSorted(contents, sortTypes[columns[currentColumn].id], columns[currentColumn].sort);
if (!currentColumnOrder)
currentColumnOrder = true;

Powered by Google App Engine
This is Rietveld 408576698