| 07:14 | <MikeSmith> | Ms2ger: any idea why http://w3c-test.org/web-platform-tests/master/dom/interfaces.html doesn't run to completion? |
| 07:15 | <Ms2ger> | It doesn't? |
| 07:15 | <MikeSmith> | there are 3983 tests in the source |
| 07:15 | <MikeSmith> | in Firefox and Chrome for me, both run only 1331 tests and stop |
| 07:15 | <Ms2ger> | How did you count? :) |
| 07:16 | <MikeSmith> | ran it locally with http://web-platform.test:8000/html/dom/interfaces.html |
| 07:17 | <MikeSmith> | and if you examine the restults results from w3c-test.org you'll a bunch of tests from teh source are not reported |
| 07:18 | <Ms2ger> | Such as? |
| 07:18 | <MikeSmith> | all of the HTMLInputElement tests |
| 07:19 | <Ms2ger> | Ah |
| 07:19 | <Ms2ger> | You're looking at the wrong test |
| 07:19 | <MikeSmith> | maybe in fact all of the HTML*Element tests |
| 07:19 | <MikeSmith> | oh |
| 07:19 | <Ms2ger> | http://w3c-test.org/web-platform-tests/master/dom/interfaces.html != http://w3c-test.org/web-platform-tests/master/html/dom/interfaces.html |
| 07:19 | <MikeSmith> | fuuuuu |
| 07:19 | <MikeSmith> | sorry for the noise |
| 07:19 | <Ms2ger> | Np |
| 07:19 | <Ms2ger> | Not sure how to make this more obvious |
| 07:22 | <MikeSmith> | I think it's pretty clear to anybody who's being careful and paying attention |
| 07:23 | <MikeSmith> | plus one tab says "DOM4 IDL tests" and the other says "HTML IDL tests" |
| 07:23 | <MikeSmith> | etc. |
| 07:25 | <Ms2ger> | Even then, you're not the first to be confused :) |
| 07:31 | <MikeSmith> | could just add <h1>DOM4 IDL tests</h1> to the source before <div id=log> |
| 07:31 | <Ms2ger> | Fair |
| 07:31 | <Ms2ger> | Let me go and do that |
| 07:33 | Ms2ger | sees https://github.com/w3c/web-platform-tests/pull/368 landed |
| 07:33 | <Ms2ger> | zcorpan++ |
| 07:33 | <Ms2ger> | denis++ |
| 07:45 | <MikeSmith> | ah cool |
| 07:45 | <MikeSmith> | having Denis working on tests is going to be a big win |
| 07:49 | <Ms2ger> | https://github.com/w3c/web-platform-tests/pull/388 |
| 08:00 | MikeSmith | looks |
| 08:01 | <MikeSmith> | btw, I wonder if wptserve can handle the kind of case that abarth describes in https://groups.google.com/a/chromium.org/d/msg/blink-dev/fnWX8n6JrTg/qw1OC_P_AlEJ |
| 08:01 | <MikeSmith> | "requires sending broken unicode on the wire and controlling exactly when the underlying socket is flushed" |
| 08:05 | <MikeSmith> | Ms2ger: how come Critic didn't create a review for https://github.com/w3c/web-platform-tests/pull/388 ? |
| 08:05 | <Ms2ger> | No idea |
| 08:06 | <Ms2ger> | jgraham? |
| 08:07 | <MikeSmith> | Ms2ger: how come you're using /resources/WebIDLParser.js instead of /resources/webidl2/lib/webidl2.js ? |
| 08:08 | <MikeSmith> | WebIDLParser.js doesn't actually exist in the repo |
| 08:08 | <Ms2ger> | That's the one that works everywhere at this point |
| 08:08 | <Ms2ger> | Except in the repo, unfortunately |
| 08:10 | <MikeSmith> | by "everywhere" you mean only some places? because it doesn't work on the jgraham/python branch with wptserve |
| 08:12 | <Ms2ger> | I thought that was going to get fixed |
| 08:13 | <MikeSmith> | oh |
| 08:13 | <MikeSmith> | I notice you used the passive voice there |
| 08:13 | <jgraham> | MikeSmith: wptserve should be able to do that. And I thought I fixed the idlharness thing but maybe not |
| 08:14 | <jgraham> | will check thst pr later |
| 08:14 | <MikeSmith> | ok |
| 08:15 | <MikeSmith> | Ms2ger: https://github.com/w3c/web-platform-tests/pull/388 LGTM as long as you guys know the /resources/WebIDLParser.js is OK |
| 08:15 | <MikeSmith> | but I been using /resources/webidl2/lib/webidl2.js in IDL tests I wrote so I guess I should change that in my tests |
| 08:17 | <Ms2ger> | It seems to be OK for the DOM / HTML tests ;) |
| 08:21 | <MikeSmith> | ah sorry yeah I mispoke |
| 08:21 | <MikeSmith> | it does work in jgraham/python too |
| 08:22 | <jgraham> | OK, good |
| 08:22 | <jgraham> | and I just fixed critic which hit a known bug |
| 08:23 | <MikeSmith> | cool |
| 08:27 | <Ms2ger> | On another note, I've found a way to reduce the review backlog: https://github.com/w3c/web-platform-tests/pull/389 |
| 08:57 | <MikeSmith> | yeah |
| 08:58 | <MikeSmith> | are those tests that plh added? |
| 08:58 | <MikeSmith> | weren't some really old crufty DOm tests imported in at some point? |
| 08:59 | <MikeSmith> | ah I see NIST listed in those W3C/domhtml/HTML* tests |
| 09:06 | <Ms2ger> | Yeah, the NIST ones |
| 09:07 | <Ms2ger> | With the extremely readable code that results from autogenerating Java and JS tests from XML files |
| 09:19 | <MikeSmith> | yeah we should dump those |
| 16:59 | <jgraham> | I didn't realise that the history navigation stuff was supposed to be GC-dependent. That seems unfortunate |
| 17:00 | <gsnedders> | jgraham: Is there any odd of you picking up the review/PR of your tests? :P |
| 17:00 | <gsnedders> | jgraham: Speaking of history navigation |
| 17:01 | <jgraham> | I don't think I can review my own tests :) |
| 17:01 | <jgraham> | But maybe I can convince someone to review them when they are workong on Servo |
| 17:03 | <gsnedders> | jgraham: Not so much review, but deal with any feedback, etc. |
| 17:03 | <gsnedders> | I don't really have the time or motivation :) |
| 17:11 | <jgraham> | I can probably do that |
| 18:46 | <rektide> | does a script when executing have any context information available telling it what it's context is? |
| 18:47 | <rektide> | how does/can a script find it' origin in the dhtml? |