Chromium Code Reviews| Index: LayoutTests/webmidi/permission.html |
| diff --git a/LayoutTests/webmidi/permission.html b/LayoutTests/webmidi/permission.html |
| index d2b1625e282c64d2600fcde756e6b6b0c3b7b84f..22c0c3bf5c0320c76d41b44d4bd9a086b08f5a0b 100644 |
| --- a/LayoutTests/webmidi/permission.html |
| +++ b/LayoutTests/webmidi/permission.html |
| @@ -8,7 +8,11 @@ |
| description("Test if sysex permission request is handled."); |
| -shouldBeDefined("testRunner.setMIDISysExPermission"); |
| +// FIXME: remove following workaround once testRunner in Chromium has a new interface. |
| +if (!testRunner.setMIDISysexPermission) |
| + testRunner.setMIDISysexPermission = testRunner.setMIDISysExPermission; |
| + |
| +shouldBeDefined("testRunner.setMIDISysexPermission"); |
| shouldBeDefined("navigator.requestMIDIAccess"); |
| window.jsTestIsAsync = true; |
| @@ -19,7 +23,7 @@ var finishSuccessfully = function () { |
| } |
| var rejectSysEx = function (next) { |
| - testRunner.setMIDISysExPermission(false); |
| + testRunner.setMIDISysexPermission(false); |
| promise = navigator.requestMIDIAccess({sysex: true}); |
| shouldBeDefined("promise"); |
| shouldBeDefined("promise.then"); |
| @@ -36,12 +40,16 @@ var rejectSysEx = function (next) { |
| } |
| var acceptSysEx = function (next) { |
| - testRunner.setMIDISysExPermission(true); |
| + testRunner.setMIDISysexPermission(true); |
| promise = navigator.requestMIDIAccess({sysex: true}); |
| shouldBeDefined("promise"); |
| shouldBeDefined("promise.then"); |
| - promise.then(function(access) { |
| + promise.then(function(a) { |
|
kouhei (in TOK)
2014/03/25 00:23:02
Optional Nit: a => obtainedAccess or something rea
Takashi Toyoshima
2014/03/25 04:08:38
Done.
|
| testPassed("sysex permission request is successfully accepted."); |
| + access = a; |
| + shouldBeDefined("access"); |
| + shouldBeDefined("access.sysexEnabled"); |
| + shouldBeTrue("access.sysexEnabled"); |
| if (next) |
| next(); |
| else |