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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm

Issue 2446483002: Import wpt@c5a14f553cba5f197743b9af605a84eddd8692a2 (Closed)
Patch Set: 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/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm
diff --git a/third_party/WebKit/LayoutTests/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm b/third_party/WebKit/LayoutTests/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm
index e74e66072104d327683aec1bf674767d3253c2a5..258eed05bf8ccbb21ba9f76ce91c998cec0f6b20 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm
+++ b/third_party/WebKit/LayoutTests/imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm
@@ -1,19 +1,30 @@
<!DOCTYPE html>
<meta charset="utf-8">
-<title>Web Workers: SharedWorker - throw URLMismatchError</title>
+<title>Web Workers: SharedWorker - same name, different URL</title>
<link rel="author" title="Intel" href="http://www.intel.com">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<div id="log"></div>
<script>
+var counter = 0
+function maybeDone() {
+ if(counter) {
+ done()
+ }
+ counter++
+}
- test(function() {
- var worker = new SharedWorker('shared-worker.js', 'name');
-
- assert_throws("URLMismatchError", function() {
- new SharedWorker('some-other-url.js', 'name');
- });
-
- }, "Create SharedWorker with different URLs but same name");
+var worker = new SharedWorker('shared-worker.js', 'name');
+worker.port.postMessage("trigger a response")
+worker.port.onmessage = (e) => {
+ assert_equals(e.data, "ping")
+ maybeDone()
+}
+// This used to throw "URLMismatchError", but the standard changed
+var worker2 = new SharedWorker('1', 'name');
+worker2.port.onmessage = (e) => {
+ assert_array_equals(e.data, ["1", "name"])
+ maybeDone()
+}
</script>

Powered by Google App Engine
This is Rietveld 408576698