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

Unified Diff: third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html

Issue 2393513004: Convert app banners to use Mojo. (Closed)
Patch Set: Add TODO Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html
diff --git a/third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html b/third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html
index 5286e4e540d4dd965e6d27bec6863ec7e9a493ae..cc49e7243d64eeedf7a427bf21f0da38cb3212dd 100644
--- a/third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html
+++ b/third_party/WebKit/LayoutTests/app_banner/app-banner-event-dispatch.html
@@ -47,14 +47,14 @@ function banner_test(index) {
});
window.addEventListener('beforeinstallprompt', event_handler);
- testRunner.dispatchBeforeInstallPromptEvent(index, ['foo', 'bar'], t.step_func(function(result) {
+ testRunner.dispatchBeforeInstallPromptEvent(['foo', 'bar'], t.step_func(function(result) {
assert_equals(result, test_case.cancel, 'Event canceled');
if (test_case.cancel) {
banner_test(index + 1);
t.done();
return;
}
- testRunner.resolveBeforeInstallPromptPromise(index, test_case.platform);
+ testRunner.resolveBeforeInstallPromptPromise(test_case.platform);
}));
}, test_case.name);
}

Powered by Google App Engine
This is Rietveld 408576698