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

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

Issue 320753002: Support javascript gtests in an extension background page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixup comment Created 6 years, 6 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 // Include test fixture.
6 GEN_INCLUDE(['../../../chromevox/testing/chromevox_unittest_base.js']);
7
8 /**
9 * Test fixture for cvox2.Background.
10 * @constructor
11 * @extends {ChromeVoxUnitTestBase}
12 */
13 function BackgroundTest() {}
14
15 BackgroundTest.prototype = {
16 __proto__: ChromeVoxUnitTestBase.prototype,
17
18 /** @override */
19 browsePreload: null,
Peter Lundblad 2014/06/11 16:45:04 This seems like something we would want in a commo
Peter Lundblad 2014/06/11 16:45:04 This seems like something we would want in a commo
Peter Lundblad 2014/06/11 16:45:04 This seems like something we would want in a commo
David Tseng 2014/06/11 17:52:54 Will refactor in a future cl (once there's a secon
20
21 /** @override */
22 closureModuleDeps: [],
Peter Lundblad 2014/06/11 16:45:04 This is the value from the base class, why do you
Peter Lundblad 2014/06/11 16:45:04 This is the value from the base class, why do you
Peter Lundblad 2014/06/11 16:45:05 This is the value from the base class, why do you
David Tseng 2014/06/11 17:52:54 Done.
23
24 /** @override */
25 testGenCppIncludes: function() {
26 GEN_BLOCK(function() {/*!
Peter Lundblad 2014/06/11 16:45:04 nit: indentation.
Peter Lundblad 2014/06/11 16:45:05 nit: indentation.
Peter Lundblad 2014/06/11 16:45:05 nit: indentation.
David Tseng 2014/06/11 17:52:54 Done.
27 #include "ash/accessibility_delegate.h"
28 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
29 #include "chrome/common/extensions/extension_constants.h"
30 */});
31 },
32
33 /** @override */
34 testGenPreamble: function() {
35 GEN_BLOCK(function() {/*!
Peter Lundblad 2014/06/11 16:45:04 nit: indentation.
Peter Lundblad 2014/06/11 16:45:04 nit: indentation.
Peter Lundblad 2014/06/11 16:45:05 nit: indentation.
David Tseng 2014/06/11 17:52:54 Done.
36 if (!chromeos::AccessibilityManager::Get()->IsSpokenFeedbackEnabled()) {
37 chromeos::AccessibilityManager::Get()->EnableSpokenFeedback(
38 true, ash::A11Y_NOTIFICATION_NONE);
39 WaitFor(extension_misc::kChromeVoxExtensionId);
40 }
41 */});
42 }
43 };
Peter Lundblad 2014/06/11 16:45:04 I think this is a candidate for refactoring to spl
Peter Lundblad 2014/06/11 16:45:05 I think this is a candidate for refactoring to spl
Peter Lundblad 2014/06/11 16:45:05 I think this is a candidate for refactoring to spl
David Tseng 2014/06/11 17:52:54 Same as above (needed only when there's 2+ files).
44
45 /** Tests ChromeVox classic is in this context. */
46 TEST_F('BackgroundTest', 'ClassicNamespaces', function() {
47 assertEquals('object', typeof(cvox));
48 assertEquals('function', typeof(cvox.ChromeVoxBackground));
49 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698