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

Unified Diff: chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs

Issue 584313003: Enable runtime switching between ChromeVox and ChromeVox next via command line. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs b/chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs
index 74abf72bf972b11e7f06d1af5494fb2e6cece00b..32a858701335f859233eafa9c2fcf724c543b42d 100644
--- a/chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs
+++ b/chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.extjs
@@ -15,6 +15,24 @@ function BackgroundTest() {}
BackgroundTest.prototype = {
__proto__: ChromeVoxE2ETest.prototype,
+ // This method is called without a |this| instance bound.
+ /** @override */
+ testGenCppIncludes: function() {
+ ChromeVoxE2ETest.prototype.testGenCppIncludes.call(this);
+ GEN('#include "base/command_line.h"');
+ GEN('#include "chromeos/chromeos_switches.h"');
+ },
+
+ // This method is called without a |this| instance bound.
+ /** @override */
+ testGenPreamble: function() {
+ GEN_BLOCK(function() {/*!
+ CommandLine* command_line = CommandLine::ForCurrentProcess();
+ command_line->AppendSwitch(chromeos::switches::kEnableChromeVoxNext);
+ */});
+ ChromeVoxE2ETest.prototype.testGenPreamble.call(this);
+ },
Peter Lundblad 2014/09/23 09:06:20 This looks like boilerplate that should go into a
David Tseng 2014/09/23 17:16:31 Once we have a second consumer class of this, I'll
+
/** @override */
setUp: function() {
this.mockTts = new MockTts();

Powered by Google App Engine
This is Rietveld 408576698