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

Unified Diff: chrome/browser/ui/webui/app_list/start_page_browsertest.js

Issue 874563003: Remove old HTML app list start page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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/browser/ui/webui/app_list/start_page_browsertest.js
diff --git a/chrome/browser/ui/webui/app_list/start_page_browsertest.js b/chrome/browser/ui/webui/app_list/start_page_browsertest.js
index b68f5906832669105c1f11de4a2f207342439259..9cdecbb31137d069c26dcb50514afc03978e5179 100644
--- a/chrome/browser/ui/webui/app_list/start_page_browsertest.js
+++ b/chrome/browser/ui/webui/app_list/start_page_browsertest.js
@@ -37,23 +37,6 @@ AppListStartPageWebUITest.prototype = {
browsePreload: 'chrome://app-list/',
/**
- * Recommend apps data.
- * @private
- */
- recommendedApps_: [
- {
- 'appId': 'app_id_1',
- 'textTitle': 'app 1',
- 'iconUrl': 'icon_url_1'
- },
- {
- 'appId': 'app_id_2',
- 'textTitle': 'app 2',
- 'iconUrl': 'icon_url_2'
- }
- ],
-
- /**
* Placeholder for mock speech recognizer.
*/
speechRecognizer: null,
@@ -126,9 +109,7 @@ AppListStartPageWebUITest.prototype = {
'launchApp',
'setSpeechRecognitionState',
'speechResult']);
- this.mockHandler.stubs().initialize().will(callFunction(function() {
- appList.startPage.setRecommendedApps(this.recommendedApps_);
- }.bind(this)));
+ this.mockHandler.stubs().initialize();
this.mockHandler.stubs().launchApp(ANYTHING);
this.registerMockSpeechRecognition_();
@@ -140,22 +121,6 @@ AppListStartPageWebUITest.prototype = {
TEST_F('AppListStartPageWebUITest', 'Basic', function() {
assertEquals(this.browsePreload, document.location.href);
-
- var recommendedApp = $('start-page').querySelector('.recommended-apps');
- assertEquals(this.recommendedApps_.length, recommendedApp.childElementCount);
- for (var i = 0; i < recommendedApp.childElementCount; ++i) {
- assertEquals(this.recommendedApps_[i].appId,
- recommendedApp.children[i].appId);
- }
-});
-
-TEST_F('AppListStartPageWebUITest', 'ClickToLaunch', function() {
- var recommendedApp = $('start-page').querySelector('.recommended-apps');
- for (var i = 0; i < recommendedApp.childElementCount; ++i) {
- this.mockHandler.expects(once()).launchApp(
- [this.recommendedApps_[i].appId]);
- cr.dispatchSimpleEvent(recommendedApp.children[i], 'click');
- }
});
TEST_F('AppListStartPageWebUITest', 'SpeechRecognitionState', function() {
« no previous file with comments | « chrome/browser/ui/app_list/start_page_service.cc ('k') | chrome/browser/ui/webui/app_list/start_page_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698