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

Side by Side Diff: chrome/test/data/extensions/api_test/managedMode/test.js

Issue 9566007: Initial Managed Mode extension API, supporting querying the setting and a stub for enabling the mod… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fixed API test.js Created 8 years, 9 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
Property Changes:
Added: svn:eol-style
+ LF
OLDNEW
(Empty)
1 // Copyright (c) 2012 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 // Managed Mode API test
6 // Run with browser_tests --gtest_filter=ExtensionApiTest.ManagedModeApi
7
8 var managedMode = chrome.experimental.managedMode;
9
10 function expect(expected, message) {
11 return chrome.test.callbackPass(function(value) {
12 chrome.test.assertEq(expected, value, message);
13 });
14 }
15
16
Finnur 2012/03/05 10:45:02 nit: remove extra line break?
17 chrome.test.runTests([
18 function disabledByDefault() {
19 managedMode.get(expect({value: false},
20 'Managed mode should be disabled.'));
21 },
22
23 // This will need to be modified once entering managed mode requires user
24 // confirmation.
25 function enterSucceeds() {
Finnur 2012/03/05 10:45:02 Wording nit: enterSucceeds -> enteringSucceeds ?
26 managedMode.enter(expect({success: true},
27 'Managed mode should be able to be entered.'));
Finnur 2012/03/05 10:45:02 Wording nit: 'Should be able to enter managed mode
28
29 managedMode.get(expect({value: true},
30 'Managed mode should be on once it has been entered.'));
31 },
32
33 function enterTwice() {
34 managedMode.enter(expect({success: true},
35 'Managed mode should be able to be entered.'));
Finnur 2012/03/05 10:45:02 Ditto? Also, indentation (and on line 37).
36 managedMode.enter(expect({success: true},
37 'Managed mode should be able to be entered twice.'));
Finnur 2012/03/05 10:45:02 Wording nit: 'Should be able to enter managed mode
38 }
39 ]);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698