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

Side by Side Diff: third_party/WebKit/LayoutTests/webshare/resources/mock-share-service.js

Issue 2703333002: Web Share: Refactor Mojo interface to now return an enum error. (Closed)
Patch Set: Created 3 years, 10 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
1 'use strict'; 1 'use strict';
2 2
3 let ShareError;
Sam McNally 2017/02/21 23:02:40 This is ugly. Consider passing webshare into the s
Matt Giuca 2017/02/23 04:18:37 Done. (I find that uglier... but whatever.)
4
3 let mockShareService = loadMojoModules( 5 let mockShareService = loadMojoModules(
4 'mockShareService', 6 'mockShareService',
5 ['mojo/public/js/bindings', 7 ['mojo/public/js/bindings',
6 'third_party/WebKit/public/platform/modules/webshare/webshare.mojom', 8 'third_party/WebKit/public/platform/modules/webshare/webshare.mojom',
7 ]).then(mojo => { 9 ]).then(mojo => {
8 let [bindings, webshare] = mojo.modules; 10 let [bindings, webshare] = mojo.modules;
9 11
12 ShareError = webshare.ShareError;
13
10 class MockShareService { 14 class MockShareService {
11 constructor(interfaceProvider) { 15 constructor(interfaceProvider) {
12 this.bindingSet_ = new bindings.BindingSet(webshare.ShareService); 16 this.bindingSet_ = new bindings.BindingSet(webshare.ShareService);
13 17
14 interfaceProvider.addInterfaceOverrideForTesting( 18 interfaceProvider.addInterfaceOverrideForTesting(
15 webshare.ShareService.name, 19 webshare.ShareService.name,
16 handle => this.bindingSet_.addBinding(this, handle)); 20 handle => this.bindingSet_.addBinding(this, handle));
17 } 21 }
18 22
19 // Returns a Promise that gets rejected if the test should fail. 23 // Returns a Promise that gets rejected if the test should fail.
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 return new Promise(resolve => { 72 return new Promise(resolve => {
69 function onKeyPress() { 73 function onKeyPress() {
70 document.removeEventListener('keypress', onKeyPress, false); 74 document.removeEventListener('keypress', onKeyPress, false);
71 resolve(functionCalledOnKeyPress()); 75 resolve(functionCalledOnKeyPress());
72 } 76 }
73 document.addEventListener('keypress', onKeyPress, false); 77 document.addEventListener('keypress', onKeyPress, false);
74 78
75 eventSender.keyDown(' ', []); 79 eventSender.keyDown(' ', []);
76 }); 80 });
77 } 81 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698