Update client side navigator.connect API to use ServicePortCollection [3/3]

LayoutTests/http/tests/navigatorconnect/resources/connect-tests.js

 // Runs various navigator.connect tests, verifying that connections are accepted
 // and rejected in the right circumstances.
 // |connect_method| is a function taking a test object as well as the regular
 // navigator.connect parameters. This makes it possible to run the same tests
 // while wrapping all navigator.connect calls in a helper making the actual
 // connection from a cross origin iframe, or a web worker.
 function run_connect_tests(connect_method) {
   promise_test(function(t) {
       return assert_promise_rejects(
         connect_method(t, 'https://example.test/service/does/not/exists'),
@@ skipping 35 matching lines @@
       var scope = sw_scope + '/accepting';
       var sw_url = 'resources/accepting-worker.js';
       return service_worker_unregister_and_register(t, sw_url, scope)
         .then(function(registration) {
             return wait_for_state(t, registration.installing, 'activated');
           })
         .then(function() {
             return connect_method(t, scope + '/service');
           })
         .then(function(port) {
-            assert_class_string(port, 'MessagePort');
+            var targetURL = new URL(port.targetURL);
+            assert_equals(targetURL.pathname, base_path() + scope + '/service');
             return service_worker_unregister(t, scope);
           });
     }, 'Connection succeeds if service worker accepts connection event.');
 
   promise_test(function(t) {
       var scope = sw_scope + '/async-rejecting';
       var sw_url = 'resources/async-connect-worker.js';
       return assert_promise_rejects(
         service_worker_unregister_and_register(t, sw_url, scope)
           .then(function(registration) {
@@ skipping 10 matching lines @@
       var scope = sw_scope + '/async-accepting';
       var sw_url = 'resources/async-connect-worker.js';
       return service_worker_unregister_and_register(t, sw_url, scope)
         .then(function(registration) {
             return wait_for_state(t, registration.installing, 'activated');
           })
         .then(function() {
             return connect_method(t, scope + '/service?accept');
           })
         .then(function(port) {
-            assert_class_string(port, 'MessagePort');
+            var targetURL = new URL(port.targetURL);
+            assert_equals(targetURL.pathname, base_path() + scope + '/service');
             return service_worker_unregister(t, scope);
           });
     }, 'Connection succeeds if service worker accepts connection event async.');
+
+  promise_test(function(t) {
+      var scope = sw_scope + '/accepting';
+      var sw_url = 'resources/accepting-worker.js';
+      return service_worker_unregister_and_register(t, sw_url, scope)
+        .then(function(registration) {
+            return wait_for_state(t, registration.installing, 'activated');
+          })
+        .then(function() {
+            return connect_method(t, scope + '/service', {name: 'somename', data: 'somedata'});
+          })
+        .then(function(port) {
+            var targetURL = new URL(port.targetURL);
+            assert_equals(targetURL.pathname, base_path() + scope + '/service');
+            assert_equals(port.name, 'somename');
+            assert_equals(port.data, 'somedata');
+            return service_worker_unregister(t, scope);
+          });
+    }, 'Returned port has correct name and data fields');
 }