| 12:08 | <zcorpan> | 0 notifications!! (down from something like 550) |
| 19:13 | <wanderview> | jugglinmike: where did we come to a decision on returning basic for same-origin no-cors requests? https://github.com/w3c/web-platform-tests/blob/master/service-workers/service-worker/fetch-response-taint.https.html#L122 |
| 19:13 | <wanderview> | I can't find it |
| 19:15 | <jugglinmike> | wanderview: https://github.com/whatwg/fetch/issues/535 |
| 19:15 | <wanderview> | thanks |
| 19:31 | <jugglinmike> | no problem |
| 20:19 | <wanderview> | jugglinmike: does chrome actually pass this somehow? https://github.com/w3c/web-platform-tests/blame/master/service-workers/service-worker/resources/fetch-response-xhr-iframe.https.html#L43 |
| 20:19 | <wanderview> | seems that should throw since event doesn't exist |
| 20:20 | <wanderview> | jugglinmike: I'll fix it locally and get it upstreamed |
| 20:22 | <jugglinmike> | wanderview: it does pass |
| 20:22 | <jugglinmike> | but that change is definitely not intention |
| 20:22 | <jugglinmike> | intentional |
| 20:22 | <jugglinmike> | It passes because of this https://developer.mozilla.org/en-US/docs/Web/API/Window/event |
| 20:23 | <wanderview> | ah.... |
| 20:23 | <wanderview> | ok |
| 20:23 | <wanderview> | I didn't know that existed |
| 20:23 | <jugglinmike> | Sometimes |
| 20:27 | <wanderview> | jugglinmike: I guess we are thinking of implementing it in the future... but some stuff with shadow dom needs to be spec'd |
| 20:27 | <wanderview> | or something |
| 20:31 | <jugglinmike> | the lack of support in Firefox is a good thing. I'd like that to go away, so FF proves that interop isn't a problem |
| 20:38 | <wanderview> | jugglinmike: does this one pass for chrome? https://github.com/w3c/web-platform-tests/blob/master/service-workers/service-worker/fetch-request-fallback.https.html#L96 |
| 20:38 | <wanderview> | AFAICT the harness does not like passing null for the exception type there |
| 20:38 | <wanderview> | https://github.com/w3c/web-platform-tests/blob/51a11c7f17d967633edbefff36cfb2f0ae23c074/resources/testharness.js#L1182 |
| 20:43 | <jugglinmike> | wanderview: You're right; it does not. We were only able to land it because of timing issues in the upstreaming process. Mek caught that her https://codereview.chromium.org/2858933003#msg23 |
| 20:43 | <jugglinmike> | and while I offered to "correct this in WPT directly", I failed to follow through on that offer |
| 20:43 | <jugglinmike> | I will do so presently |
| 20:44 | <wanderview> | jugglinmike: I can do it... but what did you end up using? |
| 20:44 | <wanderview> | 'NetworkError' or new TypeError()? |
| 20:45 | <jugglinmike> | Oh, hey |
| 20:45 | <jugglinmike> | Actually, I'm not as big of a screw-up as I thought |
| 20:45 | <jugglinmike> | Here's the patch https://github.com/w3c/web-platform-tests/pull/5869 |
| 20:46 | <jugglinmike> | I can't request your review on GitHub.com, though. Do you have commit rights to WPT? |
| 20:47 | <wanderview> | no |
| 20:47 | <wanderview> | I can land it locally and get it upstreamed if you want |
| 20:48 | <wanderview> | jugglinmike: that patch fixes the test for us |
| 20:48 | <wanderview> | jugglinmike: I'm doing a pass of WPT stuff and will request a sync next week sometime |
| 20:49 | <wanderview> | or you can land it there and we'll get it on sync |
| 20:49 | <wanderview> | I guess it doesn't matter |
| 20:50 | <jugglinmike> | Unfortunately, it's out of my hands now |
| 20:50 | <wanderview> | oh? |
| 20:50 | <jugglinmike> | It needs a review |
| 20:51 | <jugglinmike> | I'll add Matt on the Chromium team to the list of requested reviewers. He's in Japan, but he may be able to get to it before we're in on Monday morning |
| 20:51 | <wanderview> | jugglinmike: do you mind if I land it locally and upstream that way? |
| 20:51 | <wanderview> | ok, I'll just wait |
| 20:58 | <wanderview> | jugglinmike: can you request review from me now? |
| 20:58 | <wanderview> | oh, mek approved them |
| 20:59 | <jugglinmike> | hah, seems he did |
| 20:59 | <jugglinmike> | Thanks, Mek! |
| 20:59 | <wanderview> | will it auto-merge? |
| 21:00 | <jugglinmike> | Refresh |
| 21:00 | <jugglinmike> | It's merged now, too |
| 21:00 | <wanderview> | awesome, thanks |
| 21:00 | <wanderview> | I have trouble reading |
| 21:03 | <jugglinmike> | No worries. It's the end of the week, after all |
| 21:17 | <smaug> | rniwa: webkit doesn't support HTML Imports , right? |
| 21:17 | <smaug> | and doesn't plan to |
| 21:17 | <rniwa> | smaug: nope |
| 21:18 | <rniwa> | smaug: we don't support HTML imports or builtin custom elements, and we don't plan to |
| 21:18 | <smaug> | wanderview: ^ |
| 21:24 | <wanderview> | thanks |
| 21:24 | <wanderview> | rniwa: do you guys have any thoughts on "html modules" approach? I've heard some rumblings about that |
| 21:24 | <yoav> | Domenic: Reviewed https://github.com/whatwg/html/pull/2588 |
| 21:25 | <yoav> | The CL with the tests is ready to go. Ideally we want to land both at the same time |
| 21:25 | <yoav> | (and I don't have write access) |
| 21:33 | <Domenic> | yoav: should be good to go, except you gotta merge the preload spec change :) |
| 21:36 | <yoav> | Domenic: deal! |
| 21:37 | <Domenic> | yoav: https://github.com/w3c/preload/issues/95 and https://github.com/w3c/preload/issues/80 now seem closable |
| 21:37 | <yoav> | just closed |
| 21:38 | <Domenic> | :) |
| 21:39 | <yoav> | Next is https://codereview.chromium.org/2903653005/. I'll change the IDL according to your comments and land |
| 21:41 | <rniwa> | wanderview: html modules seem more promising but we haven't really hashed out details ourselves yet |
| 21:52 | <intervalia> | The site `https://github.com/whatwg` says "To be invited, just ask us on IRC; all are welcome to participate." SO I am wondering if I can be added. |
| 21:53 | <intervalia> | Also, how do I submit a proposal for things related to V1 Web Components? |
| 21:56 | <Domenic> | intervalia: file an issue on https://github.com/w3c/webcomponents/, but please pay attention to https://wiki.whatwg.org/wiki/FAQ#Is_there_a_process_for_adding_new_features_to_a_specification.3F |
| 21:56 | <domfarolino> | intervalia: Hm I think that means more along the lines of anyone can participate, just ask and mentorship can be provided, since there are only 39 people publically associated with the org. Could be wrong though. |
| 21:56 | <Domenic> | indeed, that's what it means :) |
| 21:57 | <intervalia> | OK. Maybe a little clarification to that message... |
| 21:57 | <intervalia> | Thanks for the info on Web Components. |
| 22:05 | <domfarolino> | publicly* lol |
| 23:59 | <KiChjang_> | Domenic, i vote we make a joke about the subsubsubsubsubsubsteps littered around the spec in XHR for the next april fools |