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

Side by Side Diff: ui/webui/resources/js/webui_resource_test.js

Issue 1150173003: Fix some JS style nits. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge Created 5 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
« no previous file with comments | « ui/webui/resources/js/parse_html_subset.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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 /** 5 /**
6 * Tests that an observation matches the expected value. 6 * Tests that an observation matches the expected value.
7 * @param {Object} expected The expected value. 7 * @param {Object} expected The expected value.
8 * @param {Object} observed The actual value. 8 * @param {Object} observed The actual value.
9 * @param {string=} opt_message Optional message to include with a test 9 * @param {string=} opt_message Optional message to include with a test
10 * failure. 10 * failure.
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 assertEquals(expected, observed, opt_message); 114 assertEquals(expected, observed, opt_message);
115 } 115 }
116 } 116 }
117 117
118 /** 118 /**
119 * Defines runTests. 119 * Defines runTests.
120 */ 120 */
121 (function(exports) { 121 (function(exports) {
122 /** 122 /**
123 * List of test cases. 123 * List of test cases.
124 * @type {Array.<string>} List of function names for tests to run. 124 * @type {Array<string>} List of function names for tests to run.
125 */ 125 */
126 var testCases = []; 126 var testCases = [];
127 127
128 /** 128 /**
129 * Indicates if all tests have run successfully. 129 * Indicates if all tests have run successfully.
130 * @type {boolean} 130 * @type {boolean}
131 */ 131 */
132 var cleanTestRun = true; 132 var cleanTestRun = true;
133 133
134 /** 134 /**
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 * @param {boolean} success Indicates if the test completed successfully. 201 * @param {boolean} success Indicates if the test completed successfully.
202 */ 202 */
203 function endTests(success) { 203 function endTests(success) {
204 domAutomationController.setAutomationId(1); 204 domAutomationController.setAutomationId(1);
205 domAutomationController.send(success ? 'SUCCESS' : 'FAILURE'); 205 domAutomationController.send(success ? 'SUCCESS' : 'FAILURE');
206 } 206 }
207 207
208 window.onerror = function() { 208 window.onerror = function() {
209 endTests(false); 209 endTests(false);
210 }; 210 };
OLDNEW
« no previous file with comments | « ui/webui/resources/js/parse_html_subset.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698