Closed Bug 1339885 Opened 4 years ago Closed 4 years ago
.https .html has commented out test cases
It looks like we accidentally left some commented out test cases in clients-matchall-client-types.https.html when we imported it from blink. We won't pass these, but they shouldn't be commented out since other browsers might pass them.
This patch: 1) Makes clients.matchAll() rejections fail the test instead of timing out. 2) Separates window-only tests (which we pass) from other tests (which we fail) into different test cases. 3) Marks the second test case as expected fail.
Attachment #8837741 - Flags: review?(bzbarsky)
Comment on attachment 8837741 [details] [diff] [review] Uncomment test cases from WPT test and move to separate case that can be marked expected fail. r=bz r=me. It's too bad we can't postMessage exception values, and hence lose things like stacks. :(
Attachment #8837741 - Flags: review?(bzbarsky) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6296e675694a Uncomment test cases from WPT test and move to separate case that can be marked expected fail. r=bz
You need to log in before you can comment on or make changes to this bug.