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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector-unit/filtered-list-widget-providers.js

Issue 2916743002: [DevTools] Introduce Common.List used as a backend for list controls (Closed)
Patch Set: addressed comments Created 3 years, 7 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/http/tests/inspector-unit/filtered-list-widget-providers.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector-unit/filtered-list-widget-providers.js b/third_party/WebKit/LayoutTests/http/tests/inspector-unit/filtered-list-widget-providers.js
index 44774ddb2726dee6bd173c7884b6455f8460cee9..13b4d9834b55146eee571b3c61c9be888bff551b 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector-unit/filtered-list-widget-providers.js
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector-unit/filtered-list-widget-providers.js
@@ -42,18 +42,18 @@ function test() {
}
function dump() {
- var list = filteredListWidget._list;
if (filteredListWidget._bottomElementsContainer.classList.contains('hidden')) {
TestRunner.addResult('Output: <hidden>');
return;
}
- if (list.element.classList.contains('hidden')) {
+ if (filteredListWidget._list.element.classList.contains('hidden')) {
TestRunner.addResult('Output: ' + filteredListWidget._notFoundElement.textContent);
return;
}
+ var items = filteredListWidget._items;
var output = [];
- for (var i = 0; i < list.length(); ++i)
- output.push(filteredListWidget._provider.itemKeyAt(list.itemAtIndex(i)));
+ for (var i = 0; i < items.length(); ++i)
+ output.push(filteredListWidget._provider.itemKeyAt(items.itemAtIndex(i)));
TestRunner.addResult('Output:' + JSON.stringify(output));
}
}

Powered by Google App Engine
This is Rietveld 408576698