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

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: 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..dd596d3e084c3d35614bd90a069cc2dceeb4e47c 100644
--- a/chrome/test/data/webui/extensions/extension_sidebar_test.js
+++ b/chrome/test/data/webui/extensions/extension_sidebar_test.js
@@ -29,13 +29,10 @@ cr.define('extension_sidebar_tests', function() {
updateAllExtensions: function() {},
/** @override */
- scrollToExtensions: function() {},
+ showExtensions: function() {},
/** @override */
- scrollToApps: function() {},
-
- /** @override */
- scrollToWebsites: function() {},
+ showApps: function() {},
};
/** @enum {string} */
@@ -61,7 +58,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 +92,9 @@ 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'), 'showExtensions', []);
mockDelegate.testClickingCalls(
- sidebar.$$('#sections-websites'), 'scrollToWebsites', []);
+ sidebar.$$('#sections-apps'), 'showApps', []);
});
});
}

Powered by Google App Engine
This is Rietveld 408576698