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

Side by Side Diff: chrome/browser/resources/chromeos/chromevox/testing/fake_objects.js

Issue 484063003: Port braille related tests from ChromeVox upstream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: rebase 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
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
David Tseng 2014/09/08 21:45:59 Maybe call this file fake_event.js?
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 /**
6 * Fakes a Chrome event that supports one listener.
7 * @constructor
David Tseng 2014/09/08 21:45:59 Can you inherit from a ChromeEvent?
David Tseng 2014/09/09 20:37:48 Maybe call this class FakeChromeEvent then :).
8 */
9 function FakeEvent() {
10 /** @private {Function} */
David Tseng 2014/09/08 21:46:00 Is this valid js doc?
11 this.listener_ = null;
12 }
13
14 FakeEvent.prototype = {
15 /**
16 * Fakes the corresponding call on a Chrome event. Sets the listener and
17 * fails the test if it is already set.
David Tseng 2014/09/08 21:45:59 Why? This isn't the behavior of ChromeEvent.
18 * @param {Function} listener The new listener.
19 */
20 addListener: function(listener) {
21 this.assertNoListener();
22 this.listener_ = listener;
23 },
24
25 /**
26 * Gets the listener of the event, failing the test if there's noone.
David Tseng 2014/09/08 21:45:59 nit: none
27 * @return {Function} The event's listener.
28 */
29 getListener: function() {
David Tseng 2014/09/08 21:45:59 Is this part of the ChromeEvent interface?
30 assertNotEquals(null, this.listener_);
31 return this.listener_;
32 },
33
34 /**
35 * Asserts that this object doesn't have any listener added.
36 */
37 assertNoListener: function() {
38 assertEquals(null, this.listener_);
39 }
40 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698