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

Side by Side Diff: chrome/test/data/webui/history_browsertest.js

Issue 513923003: Fix enable a11y tests for History and fix failures (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes based on feedback Created 6 years, 3 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/resources/history/history.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 GEN('#include "chrome/test/data/webui/history_ui_browsertest.h"'); 5 GEN('#include "chrome/test/data/webui/history_ui_browsertest.h"');
6 6
7 /** @const */ var TOTAL_RESULT_COUNT = 160; 7 /** @const */ var TOTAL_RESULT_COUNT = 160;
8 /** @const */ var WAIT_TIMEOUT = 200; 8 /** @const */ var WAIT_TIMEOUT = 200;
9 9
10 /** 10 /**
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 __proto__: testing.Test.prototype, 136 __proto__: testing.Test.prototype,
137 137
138 /** 138 /**
139 * Browse to the history page & call our preLoad(). 139 * Browse to the history page & call our preLoad().
140 */ 140 */
141 browsePreload: 'chrome://history-frame', 141 browsePreload: 'chrome://history-frame',
142 142
143 /** @override */ 143 /** @override */
144 typedefCppFixture: 'HistoryUIBrowserTest', 144 typedefCppFixture: 'HistoryUIBrowserTest',
145 145
146 /** @override */
147 runAccessibilityChecks: true,
148
149 /** @override */
150 accessibilityIssuesAreErrors: true,
151
146 isAsync: true, 152 isAsync: true,
147 }; 153 };
148 154
149 /** 155 /**
150 * Fixture for History WebUI testing which returns some fake history results 156 * Fixture for History WebUI testing which returns some fake history results
151 * to the frontend. Other fixtures that want to stub out calls to the backend 157 * to the frontend. Other fixtures that want to stub out calls to the backend
152 * can extend this one. 158 * can extend this one.
153 * @extends {BaseHistoryWebUITest} 159 * @extends {BaseHistoryWebUITest}
154 * @constructor 160 * @constructor
155 */ 161 */
(...skipping 856 matching lines...) Expand 10 before | Expand all | Expand 10 after
1012 TEST_F('HistoryWebUIWithSchemesTest', 'groupingWithSchemes', function() { 1018 TEST_F('HistoryWebUIWithSchemesTest', 'groupingWithSchemes', function() {
1013 // Switch to the week view. 1019 // Switch to the week view.
1014 $('timeframe-filter-week').click(); 1020 $('timeframe-filter-week').click();
1015 waitForCallback('historyResult', function() { 1021 waitForCallback('historyResult', function() {
1016 // Each URL should be organized under a different "domain". 1022 // Each URL should be organized under a different "domain".
1017 expectEquals(document.querySelectorAll('.entry').length, 4); 1023 expectEquals(document.querySelectorAll('.entry').length, 4);
1018 expectEquals(document.querySelectorAll('.site-domain-wrapper').length, 4); 1024 expectEquals(document.querySelectorAll('.site-domain-wrapper').length, 4);
1019 testDone(); 1025 testDone();
1020 }); 1026 });
1021 }); 1027 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/history/history.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698