summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--chrome/test/data/webui/list_test.html33
-rw-r--r--chrome/test/data/webui/webui_resource_browsertest.cc12
-rw-r--r--ui/webui/resources/js/cr/ui/list.js1
3 files changed, 46 insertions, 0 deletions
diff --git a/chrome/test/data/webui/list_test.html b/chrome/test/data/webui/list_test.html
new file mode 100644
index 0000000..a9239e9
--- /dev/null
+++ b/chrome/test/data/webui/list_test.html
@@ -0,0 +1,33 @@
+<!DOCTYPE html>
+<html>
+<body>
+
+<script>
+
+function testClearPinnedItem() {
+ var list = document.createElement('ul');
+ list.style.position = 'absolute';
+ list.style.width = '800px';
+ list.style.height = '800px';
+ cr.ui.List.decorate(list);
+ document.body.appendChild(list);
+
+ var model = new cr.ui.ArrayDataModel(['Item A', 'Item B']);
+ list.dataModel = model;
+ list.selectionModel.setIndexSelected(0, true);
+ list.selectionModel.leadIndex = 0;
+ list.ensureLeadItemExists();
+
+ list.style.height = '0px';
+ model.splice(0, 1);
+
+ list.style.height = '800px';
+ list.redraw();
+ assertEquals('Item B',
+ list.querySelectorAll('li')[0].textContent);
+}
+
+</script>
+
+</body>
+</html>
diff --git a/chrome/test/data/webui/webui_resource_browsertest.cc b/chrome/test/data/webui/webui_resource_browsertest.cc
index ee49f10..acdbe2d 100644
--- a/chrome/test/data/webui/webui_resource_browsertest.cc
+++ b/chrome/test/data/webui/webui_resource_browsertest.cc
@@ -59,6 +59,18 @@ IN_PROC_BROWSER_TEST_F(WebUIResourceBrowserTest, EventTargetTest) {
RunTest(base::FilePath(FILE_PATH_LITERAL("event_target_test.html")));
}
+IN_PROC_BROWSER_TEST_F(WebUIResourceBrowserTest, ListTest) {
+ AddLibrary(IDR_WEBUI_JS_CR);
+ AddLibrary(IDR_WEBUI_JS_CR_EVENT_TARGET);
+ AddLibrary(IDR_WEBUI_JS_CR_UI);
+ AddLibrary(IDR_WEBUI_JS_CR_UI_ARRAY_DATA_MODEL);
+ AddLibrary(IDR_WEBUI_JS_CR_UI_LIST_ITEM);
+ AddLibrary(IDR_WEBUI_JS_CR_UI_LIST_SELECTION_CONTROLLER);
+ AddLibrary(IDR_WEBUI_JS_CR_UI_LIST_SELECTION_MODEL);
+ AddLibrary(IDR_WEBUI_JS_CR_UI_LIST);
+ RunTest(base::FilePath(FILE_PATH_LITERAL("list_test.html")));
+}
+
IN_PROC_BROWSER_TEST_F(WebUIResourceBrowserTest, GridTest) {
AddLibrary(IDR_WEBUI_JS_CR);
AddLibrary(IDR_WEBUI_JS_CR_EVENT_TARGET);
diff --git a/ui/webui/resources/js/cr/ui/list.js b/ui/webui/resources/js/cr/ui/list.js
index 1dc38b6..3f1d1a8 100644
--- a/ui/webui/resources/js/cr/ui/list.js
+++ b/ui/webui/resources/js/cr/ui/list.js
@@ -631,6 +631,7 @@ cr.define('cr.ui', function() {
}
}
this.cachedItems_ = newCachedItems;
+ this.pinnedItem_ = null;
var newCachedItemHeights = {};
for (var index in this.cachedItemHeights_) {