Commit 7b021dae authored by May Lippert's avatar May Lippert Committed by Commit Bot

[SplitSettings] Fix broken TTS a11y test when split-settings flag is enabled.

BUG=950007
TEST=Ran browser tests with and without flag

Change-Id: I0946762b7445ce5799043b50fdff3efd57a670f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749004
Commit-Queue: May Lippert <maybelle@chromium.org>
Reviewed-by: default avatarHector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#686559}
parent dfcd5550
...@@ -9,15 +9,31 @@ ...@@ -9,15 +9,31 @@
// SettingsAccessibilityTest fixture. // SettingsAccessibilityTest fixture.
GEN_INCLUDE([ GEN_INCLUDE([
'//chrome/test/data/webui/polymer_browser_test_base.js',
'settings_accessibility_test.js', 'settings_accessibility_test.js',
]); ]);
GEN('#include "chromeos/constants/chromeos_features.h"');
// TODO(crbug/950007): refactor this into an OSSettingsAccessibilityTest class
// eslint-disable-next-line no-var // eslint-disable-next-line no-var
var TtsAccessibilityTest = class extends SettingsAccessibilityTest { var TtsAccessibilityTest = class extends PolymerTest {
/** @override */ /** @override */
get commandLineSwitches() { get commandLineSwitches() {
return ['enable-experimental-a11y-features']; return ['enable-experimental-a11y-features'];
} }
/** @override */
get featureList() {
// Always test with SplitSettings on because the pages are the same in the
// legacy combined settings and we don't want to test everything twice.
return {enabled: ['chromeos::features::kSplitSettings']};
}
/** @override */
get browsePreload() {
return 'chrome://os-settings/';
}
}; };
AccessibilityTest.define('TtsAccessibilityTest', { AccessibilityTest.define('TtsAccessibilityTest', {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment