[Bug 24077] New: second-argument-null.html worker test is incorrect

https://www.w3.org/Bugs/Public/show_bug.cgi?id=24077

            Bug ID: 24077
           Summary: second-argument-null.html worker test is incorrect
           Product: WebAppsWG
           Version: unspecified
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Test suite
          Assignee: dave.null@w3.org
          Reporter: bzbarsky@mit.edu
        QA Contact: public-webapps-bugzilla@w3.org
                CC: mike@w3.org, public-webapps@w3.org, simonp@opera.com

The test has:

  try {
    postMessage(1, null);
  } catch(e) {
    postMessage(''+e);
  }

and asserts that 1 is posted.

But the spec for postMessage is:

  void postMessage(any message, optional sequence<Transferable> transfer);

and passing null to the second argument of that should throw, per webidl.

I recommend we do three things:

1)  Add a copy of this test that replaces the first postMessage line with:

     postMessage(1);

2)  Add a copy of this test that replaces the first postMessage line with:

     postMessage(1, undefined);

3)  Modify this test to check that postMessage(1, null) in fact throws.

#1 and #2 would test correct handling of the optional argument when it's not
actually passed and #3 would test correct handling of null here.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Received on Thursday, 12 December 2013 15:12:19 UTC