00:04
<Domenic>
Hmm yeah
00:05
<Domenic>
Not sure how this fits into the Web IDL type system
00:05
<Domenic>
Since it doesn't make much sense as a return type
00:26
<TabAtkins>
Well, it can output as a Map as a return value.
00:27
<TabAtkins>
Same as sequence<> outputs as an Array (but can be lots of things as an argument value)
00:27
<Domenic>
Hmm not bad
01:01
<Domenic>
Wow 88.6% of pages use UTF-8?? https://w3techs.com/technologies/overview/character_encoding/all
01:01
<Domenic>
I didn't know the world was such a happy place
01:09
<Domenic>
https://en.wikipedia.org/wiki/UTF-8#/media/File:Utf8webgrowth.svg
01:16
<Domenic>
I wonder why https://en.wikipedia.org/wiki/UTF-8#WTF-8 doesn't mention https://simonsapin.github.io/wtf-8/
01:19
<Domenic>
http://people.ds.cam.ac.uk/fanf2/hermes/doc/qsmtp/draft-fanf-wtf8.html is funny
03:19
<Krinkle>
Curious if there's any recommended way of dealing with localStorage key eviction, in a way that scales reasonably and performs well on mobile. https://github.com/whatwg/storage/issues/11#issuecomment-283233411 / https://phabricator.wikimedia.org/T121646#3063515
06:42
<yoav>
annevk: Finally getting back to https://github.com/whatwg/fetch/pull/442/files#r96880945
06:46
<yoav>
If I properly understand your suggestion there, you'd prefer to get rid of all changes in current PR, create the concept of "potential destination", have preload use that, and just translate is to destination before preload triggers a fetch
06:46
<yoav>
is that correct?
08:05
<JakeA>
MikeSmith: the outline stuff in the html5 checker… does it miss section roots because the generated outline doesn't provide a link from the parent to child roots?
08:09
<MikeSmith>
JakeA: no it doesn’t show the headings for the sectioning roots because that is the required result of following the outline algorithm as specified
08:10
<MikeSmith>
the spec says nothing about generating separate outlines for the sectioning roots and then linking them together
08:11
<MikeSmith>
the algorithm if applied to a document builds only a single outline for the document, which drops any headings for the sectioning roots
08:12
<MikeSmith>
all the existing implementations of the outline algorithm have followed the spec on this
08:12
<MikeSmith>
e.g., h5o and gsnedders outliner
08:14
<MikeSmith>
I guess it could be argued that the it’s assumed we’re supposed to re-run the algorithm on each sectioning root we run into and .. somehow store a separate object for each and then somehow link them together
08:14
<MikeSmith>
but that’s not what the actual normative spec text says to do
08:16
<JakeA>
MikeSmith: this sounds like *looks into camera* the root of the issue
08:16
<JakeA>
Trying to get confirmation from Hixie
08:26
<MikeSmith>
it’s cool when you look into the camera like that
15:04
<tobie>
JakeA: I'm working on motion sensors right now. Some of the use cases sound like great candidates for background events.
15:05
<tobie>
JakeA: was curious if you had data on how often SW are woken-up in practice
15:05
<tobie>
JakeA: + battery cost of doing so
15:05
<tobie>
JakeA: use cases are e.g. indoor navigation using sensor data
15:06
<JakeA>
tobie: I think the battery cost is generally pretty high, because it involves booting the browser up. I don't have data though, because no two cases are alike
15:06
<JakeA>
tobie: But I know that the Physical Web folks couldn't wake the SW up per beacon
15:07
<annevk>
So I was thinking that if we added integrity to service workers we could maybe allow them cross-origin, but that still wouldn't protect against an XSS-installed service worker
15:07
<JakeA>
That's the main reason we haven't allowed them
15:08
<annevk>
So what you really need is some kind of nonce+integrity or integrity declared through HTTP headers
15:08
<annevk>
If integrity is declared through HTTP, you should be safe from XSS
15:08
<annevk>
I'm not sure what the latest on that is exactly though
15:09
<annevk>
(Also, I'm just brainstorming a bit here, I haven't seen many requests or indication this is a priority.)
15:09
<tobie>
JakeA: so even 1/minute would be too much
15:10
<JakeA>
tobie: oh wow yes
15:11
<tobie>
JakeA: need to balance size of data collected too.
15:11
<JakeA>
tobie: I think the main cost is booting up the browser and potentially the radio too
15:21
<tobie>
JakeA: sounds like I need to revise my plans, than. :(
15:22
<tobie>
s/than/then/
15:22
<tobie>
JakeA: thanks!
15:37
<annevk>
jgraham: Ms2ger: here's an alternative WPT review plan: I land my own PRs and file browser bugs on the failures which will lead to review
15:38
<annevk>
jgraham: Ms2ger: I also think we should maybe prune OWNER files where folks haven't been reviewing changes in a while as the current system is more like a PING file since nobody can be hold accountable
15:40
<annevk>
tobie: I think for mobile sensors you really want something like Apple's M chip series
15:40
<annevk>
tobie: that just store a bunch of the data and then expose it to the app at the point the user might do something with it
15:41
<annevk>
tobie: depends a bit on the sensor too of course
15:41
<tobie>
annevk: the owner files were bootstrapped from the list of committers to that directory.
15:41
<tobie>
annevk: they were *mean't* to be edited and pruned.
15:41
<tobie>
meant
15:41
tobie
can't spell todya
15:42
<tobie>
annevk: yes, I guess. I was mostly wondering if there was a way to bridge different use cases through SW
15:44
<gsnedders>
annevk: there are definitely people who want to be in a PING file though
15:45
<gsnedders>
annevk: one option is we simply introduce two separate files and update wpt-pr-bot
15:45
<tobie>
annevk: e.g. storing motion data for mapping out the number of steps on a run once you're back from it, vs near real-time indoor positioning on map using sensors.
15:45
<annevk>
gsnedders: I think a reasonable solution would be to move all OWNERS to PING and only once we have an actual OWNER add them there
15:46
<tobie>
gsnedders: add categories to owners.
15:47
<tobie>
gsnedders: rename all the files if you want to.
15:47
<tobie>
gsnedders: but dealing with more than one file seems tedious
15:48
<gsnedders>
tobie: yeah, on the whole I agree
15:52
<Ms2ger>
OWNERS was intended to mean PING, really
15:53
<Ms2ger>
I don't think allowing particular people to land without any review is a good idea
15:53
<gsnedders>
landing without review also ties into the discussions about how to deal with specs that are still incubating and moving fast
15:57
<annevk>
Ms2ger: I didn't say it's without review, it's just that the review is delayed
15:57
<annevk>
Ms2ger: because currently there's no review and that means everything is stalled
15:58
<annevk>
And the OWNERS/PING thing was unrelated to this review thing, that was more about making sure we actually know who are responsible (or whether we lack someone responsible)
15:59
<annevk>
You need some accountability
16:00
<annevk>
We could also do OWNERS / PEERS
16:02
<Ms2ger>
The situation we have right now is that nobody is responsible
16:03
<Ms2ger>
That's a problem of resource allocation, not files
16:03
<annevk>
Well, making that transparent seems like a good first step
16:04
<annevk>
And I actually want to be responsible for URL/CORS/XHR/Fetch/Encoding
16:05
<annevk>
And have made lots of changes over the past couple weeks to get those more into shape as you may or may not have noticed
16:05
<annevk>
It's based on that experience that I'm here suggesting changes
16:05
<annevk>
Being dismissive of them without really suggesting something else is not helping
16:07
<jgraham>
annevk: But I think the problem is that no one believes that review-later will cause review to happen
16:07
<annevk>
I do?
16:07
<annevk>
Implementations will have to fix their bugs after all
16:07
<annevk>
Or do you think they won't ever fix their bugs?
16:08
<annevk>
At that point you've lost
16:08
<gsnedders>
annevk: what vendor makes a general effort to fix new failing tests in wpt? AFAIK, none seriously do.
16:08
<annevk>
Lots do when bugs are filed
16:08
<annevk>
Firefox has, WebKit has, Edge has, Chrome has
16:09
<annevk>
I know because I've been doing this for a while
16:09
<gsnedders>
so we need a system that when a new failing test is added automatically file bugs?
16:09
<annevk>
No, I just file the bugs
16:09
<annevk>
As I said earlier...
16:09
<gsnedders>
no, I mean in general case of how we get people to care about failing tests
16:10
<annevk>
They already do
16:10
<gsnedders>
do they? when Gecko, Blink sync up to the latest web-platform-tests, they add expectations files to ignore any failures but don't analyse or file bugs for them.
16:11
<annevk>
That's their current system, but if you talk to the engineers they do want to fix those failures
16:11
<annevk>
And if you file bugs they do fix them
16:12
<annevk>
Again, I have some experience with this
16:12
<gsnedders>
the problem is I don't think it's practical to get every contributor to wpt to be running tests in all browsers and filing bugs
16:13
<annevk>
Geez
16:13
<annevk>
So require review for those contributors
16:14
<annevk>
Can we talk a bit more constructively about this perhaps?
16:16
<gsnedders>
The problem is we need a way that doesn't rely on individual contributors because otherwise we'll have failures slip through. That's my point.
16:17
<gsnedders>
And the problem is a hard one unless there's someone who's job it is is to analyse all incoming failures.
16:17
<jgraham>
annevk: So I'm feeling too ill to have an argument :) But I think my issue is different from gsnedders. I don't doubt that often you can find someone to at least look at failing tests (and you, specifically, have the social capital to make that process work resaonably often). But no one is going to look at passing tests (especially if you check in a file of tests that only pass). And it seems weird that you are happy to require review for spec chang
16:17
<jgraham>
(OK, where did that get cut off?)
16:17
<gsnedders>
"review for spec change"
16:17
<jgraham>
And it seems weird that you are happy to require review for spec changes but requiring equivalent review for tests is too hard. Can't you require the same reviewer to look at both?
16:18
<annevk>
jgraham: this is cleanup of existing tests
16:18
<annevk>
jgraham: or adding coverage
16:19
<annevk>
jgraham: and I do actually review passing tests every now and then too
16:19
<annevk>
jgraham: in particular when they need to be revised or we add new features or some such
16:19
<jgraham>
annevk: That you in particular are an angel (per diveintomark) is not in question
16:19
<annevk>
gsnedders: sure, the system is not perfect, but the current system basically makes me stop contributing
16:20
<annevk>
gsnedders: which seems worse
16:20
<jgraham>
annevk: So the hacky solution is to commit to m-c and ask for review from people at Mozilla because they clearly have a job description that involves doing those reviews
16:21
<jgraham>
(and when I say "hacky" it's an explicit design goal to leverage that fact)
16:22
<jgraham>
The less hacky solution seems to be to get some individuals to agree that reviewing your cleanup tests is also part of their job. It still seems strange that you can find that group for spec changes but not for test changes
16:24
<annevk>
Sure, I can probably get them rubberstamped
16:24
<annevk>
The system not pretending OWNERS will review the test as they actually don't might help with that
16:25
<annevk>
So I don't wait several days for them and then wait several days for whatever victim I can find
16:26
<jgraham>
Sure, if you want some technical fix to not imply that OWNERS actually feel a sense of responsibility then I'm happy with that
16:52
<tobie>
annevk: fwiw: the current review process (which is to allow one r+ either in wpt or in another public system) is something I fought for when I was at FB/W3C as a way to simplify the previous process which required someone *from a different company* to approve a contribution.
16:53
<tobie>
annevk: there is nothing that prevents us from modifying that process if it turns out it's not optimum
16:53
<tobie>
annevk: I don't think there's a formal process for gathering consensus around that.
16:54
<tobie>
annevk: I'd argue filing a bug/pr on the docs where the process is specified + mailing the testing-infra@ should be enough
16:55
<tobie>
annevk: possible work arounds could include: allow people with owner status to r+ their own changes,
16:56
<tobie>
have a lits of special contributors which can r+ their own changes
16:56
<tobie>
maybe have a (light) process to get someone on that list.
16:57
<tobie>
annevk: this is not rocket science, and whatever process is in place should be there to serve the goals of the project
16:57
<tobie>
annevk: not hinder them
17:36
<annevk>
tobie: thanks for the suggestions, I guess I need to think a bit more about what exactly needs to change beyond changing OWNERS
17:58
<tobie>
annevk: thanks for the reference to Apple M-series, btw. I knew these things existed, but didn't know what they were called.
17:59
<tobie>
annevk: now I have a nice Wikipedia article to point people to.