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

Unified Diff: chrome/test/data/webui/md_bookmarks/sidebar_test.js

Issue 2823053003: [MD Bookmarks] Remove the bookmarks-sidebar element. (Closed)
Patch Set: Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/webui/md_bookmarks/sidebar_test.js
diff --git a/chrome/test/data/webui/md_bookmarks/sidebar_test.js b/chrome/test/data/webui/md_bookmarks/sidebar_test.js
index 097ba79e27d48bbffc1bb574c509606f58de95be..49e080bc14ba7894c7988030723da8d12c190ed0 100644
--- a/chrome/test/data/webui/md_bookmarks/sidebar_test.js
+++ b/chrome/test/data/webui/md_bookmarks/sidebar_test.js
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-suite('<bookmarks-sidebar>', function() {
tsergeant 2017/04/19 07:39:28 Looks like this file didn't get deleted when it sh
calamity 2017/04/26 08:03:54 Done.
tsergeant 2017/04/26 23:47:55 This file still shows up as 'M' in Rietveld instea
- var sidebar;
+suite('<bookmarks-folder-node>', function() {
+ var rootNode;
var store;
setup(function() {
@@ -24,14 +24,15 @@ suite('<bookmarks-sidebar>', function() {
});
bookmarks.Store.instance_ = store;
- sidebar = document.createElement('bookmarks-sidebar');
- replaceBody(sidebar);
+ rootNode = document.createElement('bookmarks-folder-node');
+ rootNode.itemId = '0';
+ rootNode.depth = -1;
+ replaceBody(rootNode);
Polymer.dom.flush();
});
test('selecting and deselecting folders dispatches action', function() {
- var rootFolders =
- sidebar.$['folder-tree'].querySelectorAll('bookmarks-folder-node');
+ var rootFolders = rootNode.root.querySelectorAll('bookmarks-folder-node');
var firstGen = rootFolders[0].$['descendants'].querySelectorAll(
'bookmarks-folder-node');
var secondGen =
@@ -51,8 +52,7 @@ suite('<bookmarks-sidebar>', function() {
});
test('depth calculation', function() {
- var rootFolders =
- sidebar.$['folder-tree'].querySelectorAll('bookmarks-folder-node');
+ var rootFolders = rootNode.root.querySelectorAll('bookmarks-folder-node');
var firstGen = rootFolders[0].$['descendants'].querySelectorAll(
'bookmarks-folder-node');
var secondGen =
@@ -73,8 +73,7 @@ suite('<bookmarks-sidebar>', function() {
});
test('doesn\'t highlight selected folder while searching', function() {
- var rootFolders =
- sidebar.$['folder-tree'].querySelectorAll('bookmarks-folder-node');
+ var rootFolders = rootNode.root.querySelectorAll('bookmarks-folder-node');
store.data.selectedFolder = '1';
store.notifyObservers();

Powered by Google App Engine
This is Rietveld 408576698