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

Unified Diff: chrome/test/data/webui/extensions/extension_sidebar_test.js

Issue 1913353002: [MD Extensions] Add a details subpage, move to one list (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: latest master Created 4 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/extensions/extension_sidebar_test.js
diff --git a/chrome/test/data/webui/extensions/extension_sidebar_test.js b/chrome/test/data/webui/extensions/extension_sidebar_test.js
index 71b6726e0b0af84a3773df68735e2d8052482d49..cfe5c8a8a80235864590740aaf9cee95b11a6968 100644
--- a/chrome/test/data/webui/extensions/extension_sidebar_test.js
+++ b/chrome/test/data/webui/extensions/extension_sidebar_test.js
@@ -29,13 +29,7 @@ cr.define('extension_sidebar_tests', function() {
updateAllExtensions: function() {},
/** @override */
- scrollToExtensions: function() {},
-
- /** @override */
- scrollToApps: function() {},
-
- /** @override */
- scrollToWebsites: function() {},
+ showType: function() {},
};
/** @enum {string} */
@@ -61,7 +55,7 @@ cr.define('extension_sidebar_tests', function() {
sidebar = document.querySelector('extensions-manager').sidebar;
mockDelegate = new MockDelegate();
sidebar.setDelegate(mockDelegate);
- sidebar.setScrollDelegate(mockDelegate);
+ sidebar.setListDelegate(mockDelegate);
});
test(assert(TestNames.Layout), function() {
@@ -95,11 +89,11 @@ cr.define('extension_sidebar_tests', function() {
mockDelegate.testClickingCalls(
sidebar.$$('#update-now'), 'updateAllExtensions', []);
mockDelegate.testClickingCalls(
- sidebar.$$('#sections-extensions'), 'scrollToExtensions', []);
- mockDelegate.testClickingCalls(
- sidebar.$$('#sections-apps'), 'scrollToApps', []);
+ sidebar.$$('#sections-extensions'), 'showType',
+ [extensions.ShowingType.EXTENSIONS]);
mockDelegate.testClickingCalls(
- sidebar.$$('#sections-websites'), 'scrollToWebsites', []);
+ sidebar.$$('#sections-apps'), 'showType',
+ [extensions.ShowingType.APPS]);
});
});
}
« no previous file with comments | « chrome/test/data/webui/extensions/extension_service_test.js ('k') | ui/webui/resources/polymer_resources.grdp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698