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

Unified Diff: third_party/WebKit/LayoutTests/presentation/presentation-controller-close-connection.html

Issue 2874483002: [Presentation API] Remove closed connections from PresentationConnectionList (Closed)
Patch Set: rebase with master Created 3 years, 7 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/presentation-controller-close-connection.html
diff --git a/third_party/WebKit/LayoutTests/presentation/presentation-controller-close-connection.html b/third_party/WebKit/LayoutTests/presentation/presentation-controller-close-connection.html
index 30b6e3eace704855232a5a03ab7e966e26fa72b9..0f09a27da0cf363e1aaf5a02e5269aac385fd08c 100644
--- a/third_party/WebKit/LayoutTests/presentation/presentation-controller-close-connection.html
+++ b/third_party/WebKit/LayoutTests/presentation/presentation-controller-close-connection.html
@@ -23,8 +23,6 @@ async_test(t => {
assert_equals(closeEvent.reason, "closed");
assert_equals(closeEvent.message, "");
assert_equals(connection.state, "closed");
- assert_true(receiverConnectionClosed);
- t.done();
};
connection.onconnect = () => {
// Open a receiver page and pass controller connection's
@@ -41,6 +39,7 @@ async_test(t => {
window.addEventListener("message", t.step_func(e => {
if (e.data == "receiver connection closed") {
receiverConnectionClosed = true;
+ t.done();
} else if (e.data == "receiver connection ready") {
assert_not_equals(connection, null);
connection.close();

Powered by Google App Engine
This is Rietveld 408576698