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

Unified Diff: third_party/WebKit/LayoutTests/presentation/presentationrequest.html

Issue 2148643002: [Presentation API] Adds DOMString[] constructor to PresentationRequest. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix LayoutTests Created 4 years, 5 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/presentation/presentationrequest.html
diff --git a/third_party/WebKit/LayoutTests/presentation/presentationrequest.html b/third_party/WebKit/LayoutTests/presentation/presentationrequest.html
index d4b538c2fda08c92d618cc90abb9a24c805a1769..4d16ca70510ab25ff5818208e88f8a6b7246d2a5 100644
--- a/third_party/WebKit/LayoutTests/presentation/presentationrequest.html
+++ b/third_party/WebKit/LayoutTests/presentation/presentationrequest.html
@@ -6,22 +6,41 @@
<script src="../resources/gc.js"></script>
<script>
-async_test(function(t) {
- var request = new PresentationRequest("http://example.com");
+var presentationUrl = "http://example.com";
+var presentationUrls = [presentationUrl, "cast://google.com/app_id=deadbeef"];
+
+var testUserGesture = function(t, requestArgument) {
+ var request = new PresentationRequest(requestArgument);
request.start().catch(t.step_func(function(e) {
assert_true(e instanceof DOMException);
assert_equals(e.name, "InvalidAccessError");
assert_equals(e.message, "PresentationRequest::start() requires user gesture.");
t.done();
}));
-}, "Test that the PresentationRequest.start() requires user gesture.")
+};
-test(function() {
- navigator.presentation.defaultRequest = new PresentationRequest("http://example.com");
+async_test(function(t) {
+ testUserGesture(t, presentationUrl);
+}, "Test that the PresentationRequest.start() with one URL requires user gesture.")
+
+async_test(function(t) {
+ testUserGesture(t, presentationUrls);
+}, "Test that the PresentationRequest.start() with multiple URLs requires user gesture.")
+
+var testGarbageCollection = function(requestArgument) {
+ navigator.presentation.defaultRequest = new PresentationRequest(requestArgument);
navigator.presentation.defaultRequest.onconnectionavailable = function() { };
gc();
assert_not_equals(navigator.presentation.defaultRequest.onconnectionavailable, undefined);
-}, "Test that navigator.presentation.defaultRequest.onconnectionavailable isn't reset after gc()");
+};
+
+test(function() {
+ testGarbageCollection(presentationUrl);
+}, "Test that navigator.presentation.defaultRequest.onconnectionavailable with one URL isn't reset after gc().");
+
+test(function() {
+ testGarbageCollection(presentationUrls);
+}, "Test that navigator.presentation.defaultRequest.onconnectionavailable with multiple URLs isn't reset after gc().");
</script>
</body>

Powered by Google App Engine
This is Rietveld 408576698