-
KiChjang
that's probably what's confusing the mutation events ordering
-
KiChjang
so, the behavior indicates that this script tag's script gets run *after* the document itself has loaded, and not during document parsing
-
KiChjang
that's the only way i can make sense of the behavior
-
jdm
KiChjang: any chance the script element is marked async?
-
jdm
or defer?
-
KiChjang
-
KiChjang
now it doesn't seem to me that it's been marked as async or defer
-
KiChjang
but it *might* be the case that our code thinks otherwise
-
jdm
doubtful
-
jdm
KiChjang: so what are the symptoms that you observe?
-
KiChjang
lemme grab my PR
-
KiChjang
-
crowbot
PR #21456: [WIP] Mutation observer test fix -
servo/servo #21456
-
KiChjang
lemme try and remember what i saw...
-
jdm
KiChjang: what are the text node additions that log that you refer to?
-
KiChjang
-
KiChjang
i believe the test harness appends new text node elements to this div
-
jdm
ah, right
-
jdm
KiChjang: what are the actual failures that are reported?
-
KiChjang
so, insertionStep2 expects an array of 2 elements, but it sees only 1
-
KiChjang
insertionTest2, sorry
-
KiChjang
i've looked into the mutation observer's sequence -- the only element in the array is not the event of the insertion of either inserted_script nor inserted_element
-
KiChjang
s/sequence/record
-
KiChjang
it's the text node under <div id="log">
-
jdm
KiChjang: I think it might make sense to try to make a reduced testcase that demonstrates this behaviour that you can easily run in servo and firefox and compare the output
-
Manishearth
standups: more XR integration work
-
crowbot
Status submitted successfully.
-
Manishearth
-
crowbot
Status submitted successfully.
-
ghservo
[servo] jdm opened issue #22517: Add Connection header to HTTP requests
git.io/fhviY
-
ajeffrey
standups: Spent the day working out how to make script_thread something that can execute futures.
-
crowbot
Status submitted successfully.
-
jdm
standups: rewriting a surprising number of referrer-policy tests.
-
crowbot
Status submitted successfully.
-
avadacatavra
standups: meetings, writing, etc...
-
crowbot
Status submitted successfully.
-
ghservo
[servo] Manishearth opened issue #22518: Provide wrapper for "resolve this promise on response from thread"
git.io/fhv1I
-
Manishearth
ugh why is everything terrible
-
Manishearth
the googlevr build isn't working anymore
-
Manishearth
(maybe its all android builds?)
-
ghservo
[servo] jdm opened issue #22519: Intermittent failure in /_mozilla/mozilla/task_queue_throttling.any.html
git.io/fhv1i
-
jdm
oh no
-
» jdm disembarks
-
ghservo
[webrender] kvark opened pull request #3441: Cleanup clip code a bit, improve chasing output (master...clip-cleanup)
git.io/fhvMS
-
ghservo
[servo] servo-wpt-sync opened pull request #22520: Sync WPT with upstream (20-12-2018) (master...wpt_update_20-12-2018)
git.io/fhvMx
-
taskcluster
Task "Servo daily: decision task. On failure, ping: SimonSapin, nox, emilio" complete with status 'failed'. Inspect:
tools.taskcluster.net/tasks/JRT3kT8xT6CKmO9oFceDMw
-
ghservo
[media] ferjm opened pull request #176: Expose a way to implement a backoff protocol, refactor errors and other minor improvements (master...player.src.max_buffer)
git.io/fhfUv
-
ghservo
[servo] lucasfantacuci opened pull request #22521: WIP: Use builder pattern with RequestInit (master...use_build_pattern_with_requestinit)
git.io/fhfTl
-
ghservo
[servo] ferjm opened pull request #22522: HTMLMediaElement backoff protocol and other improvements to a/v playback (master...av.playback.improvements)
git.io/fhfkL
-
ferjm
-
crowbot
PR #176: Expose a way to implement a backoff protocol, refactor errors and other minor improvements -
servo/media #176
-
crowbot
PR #22522: HTMLMediaElement backoff protocol and other improvements to a/v playback -
servo/servo #22522
-
ferjm
Manishearth: please, let me know if you don't feel confident enough reviewing the Player changes and I'll ask vjaquez to do the review after he returns from PTO
-
haedaal
Hi, I need a help to tackle assigned issue! while repeating test, it started to spit 'assert_not_equals: startTime got disallowed value 0' for all the tests. is it kind of cache problem? then how can i resolve this?
-
jdm
haedaal: you will want to figure out what in the test is causing that failure to appear, and trace that back to the code changes that you made
-
jdm
haedaal: it's possible that you have now exposed the startTime property whereas previous the test was getting undefined, now it's getting 0
-
ghservo
[rust-mozjs] bors-servo pushed 0 new commits to auto:
git.io/fhf82
-
ajeffrey
-
crowbot
PR #450: Make runtime creation safe -
servo/rust-mozjs #450
-
crowbot
PR #22491: Add basic keyboard support for magicleap -
servo/servo #22491
-
jdm
can do!
-
ajeffrey
jdm: ta
-
ghservo
[rust-mozjs] bors-servo merged auto into master:
git.io/fhf8a
-
ghservo
[rust-mozjs] bors-servo closed pull request #444: remove optional arguments from NewProxyObject (master...optional-args)
git.io/fxoQm
-
mib_96gai1
Jordan Le'Oran, Tristen Gaspadarek, Larissa Maestro, Caitlin Rose, Orono Noguchi, Timothy Shann, Mark Turner, Christopher Young, Matias S. Koedt, Marek Skiba, Sean Doherty, Kyle Gritchen
-
mib_96gai1
Zach Cox, Stephen & Leslie Braun, Mark Wetzstein, Brad Kaiserman, David Bornstein, Micah Hauptman, Stephen Engel, Dustin Milligan, Brock Williams, Jon Coplon, Ben Schnetzer, Dave Cowen, Nina Emkin
-
mib_96gai1
Ben Painter, Trevor Stoddart, Joe Ramirez, Gabriel Hart, Aaron Tap, David & Rene Pacheco, Lennon Kloser, Andrew Dunlap, Max maulitz, Glenn Schloss, Justina Adorno, Lanisha Javon Gholston
-
eijebong
jdm: Manishearth larsberg mbrubeck SimonSapin ^
-
jdm
ajeffrey: bleah, where do I find the documentation for KeyboardProperties?
-
» ajeffrey looks
-
ajeffrey
-
jdm
thanks
-
jdm
silly me for trying to find it in the local packages I had installed
-
jdm
ajeffrey: there's a "speechToTextEnabled" boolean o.O
-
ajeffrey
jdm: yup, might come in handy :)
-
ajeffrey
there's also back, forward and clear keys that I should hook up at some point.
-
ajeffrey
jdm: not sure how exokit is supporting key-down and key-up, since the lumin api only gives key-pressed
creator.magicleap.com/learn/referen…r1__0_1_1ui_1_1_keyboard_event.html
-
jdm
ajeffrey: ah, the C API exposes different things than the lumin RT API
-
jdm
-
ajeffrey
jdm: argh
-
jdm
very curious
-
ajeffrey
jdm: not sure what to do about this, I'd rather not mix-n-match C and Lumin API calls :(
-
jdm
yeah, I'm fine with what's there right now
-
ajeffrey
jdm: also, there may be different APIs for the physical and virtual keyboards.
-
ajeffrey
I'll ask on the ML forum.
-
ghservo
[webrender] lsalzman opened pull request #3442: add support for FreeType variation fonts (master...freetype-variations)
git.io/fhfaI
-
-
crowbot
PR #22493: Reduce intermittent failures for taskcluster -
servo/servo #22493
-
jdm
maybe I'll try adding hyper logging
-
ajeffrey
-
ghservo
[servo] asajeffrey opened issue #22523: Support different keyboard types in magicleap
git.io/fhfaP
-
ghservo
[servo] asajeffrey opened issue #22524: Support separate key-up and key-down events on magicleap?
git.io/fhfa7
-
eijebong
jdm: Could it be that the responses are compressed ?
-
eijebong
I still haven't taken time to fix the decoder thing...
-
jdm
eijebong: I mean, we still receive an incorrect response body in that case, right?
-
eijebong
jdm: No, we might receive nothing if the first packet is a 1 byte sized one IIRC
-
jdm
hmm
-
jdm
maybe I'll try to fix that problem today and see if anything improves
-
eijebong
I'll try to find motivation this weekend if you don't do it
-
eijebong
The switch kind of killed my motivation to do code stuff o:)
-
jdm
standups: reviews
-
crowbot
Status submitted successfully.
-
ghservo
[heapsize] est31 opened issue #103: Publish a new version of heapsize_derive
git.io/fhfrp
-
ferjm
ugh ./mach create-wpt --reftest just created a 115MB diff by changing a bunch of stuff in all MANIFEST.json files :\
-
jdm
:o
-
ferjm
anyway... it's friday and it's dinner time, so I'll investigate on monday :)
-
jdm
should be an easy fix in run_create
-
jdm
dispatch the update-manifest command first instead of adding the --manifest-update command to test-wpt
-
ferjm
cool, thanks for the pointer
-
ghservo
[servo] jdm opened issue #22525: creat-wpt mach command needs to use new manifest update implementation
git.io/fhfKk
-
» jdm watches another test job hit the 60 minute limit
-
ghservo
[rust-mozjs] longwusha opened pull request #451: [#411] Add some tests for default-initialized roots (master...master)
git.io/fhfKK
-
» jdm contemplates stealing reqwest's decoder.rs in its entirety
-
eijebong
jdm: Yeah it'd need some work to have multiple decompression algorithms but it would at least fix our problems...
-
ghservo
[rust-mozjs] bors-servo pushed 4 new commits to auto:
git.io/fhfMG
-
ghservo
rust-mozjs/auto 0c14722 lws: [#411] Add some tests for default-initialized roots
-
ghservo
rust-mozjs/auto f1ee669 lws: [#411] Use the GCMethods trait to set initial value && Complete other test cases
-
ghservo
rust-mozjs/auto b50cccf lws: [#411] merge test functions
-
ajeffrey
standups: Wrote a task executor for script_thread which allows async Rust functions to be used synchronously in JS.
github.com/asajeffrey/servo/tree/script-async-experiments
-
crowbot
Status submitted successfully.
-
lqd
ajeffrey++
-
ajeffrey
Upside: allows other pipelines in the same script thread to get on with things.
-
ajeffrey
Downside: might not be entirely spec-compliant, sigh.
servo/servo #22501
-
crowbot
Issue #22501: Use async functions in script? -
servo/servo #22501
-
» ajeffrey needs to spec-lawyer the html event loop
-
» jdm is lost in a twisty maze of types named Body and Chunk
-
eijebong
jdm: HAHAHA
-
eijebong
So sorry for you
-
» jdm takes a walk to the municipal offices instead
-
ghservo
[webrender] Eijebong opened pull request #3443: Update base64 to 0.10 (master...base64)
git.io/fhfHU
-
ajeffrey
-
ajeffrey
jdm: Any more thoughts on #22491?
-
crowbot
PR #22491: Add basic keyboard support for magicleap -
servo/servo #22491
-
pietroalbini
hi, who's managing servo's infra?
-
» ajeffrey would like to land it this year :)
-
pietroalbini
CI and bors in particular :)
-
ajeffrey
pietroalbini: jdm and Manishearth do most of the infra mgmt at the moment.
-
pietroalbini
jdm and Manishearth, just to make sure you're aware of this, travis-ci is going to migrate all the public repositories to travis-ci.com on january 31st
-
jdm
yep, I've been keeping up with it
-
jdm
ajeffrey: I'm fine with landing it
-
jdm
ajeffrey: barring the fact that almost nothing can land right now
-
ajeffrey
jdm: ta
-
pietroalbini
jdm, do you have an homu branch that works with the checks api?
-
jdm
pietroalbini: no, I guess we need to schedule time to make that happen
-
pietroalbini
jdm, yeah, we (rust infra) learned of this today by accident, it's going to be fun
-
pietroalbini
I'm probably going to look into adding the support to our fork soon btw
-
jdm
oh, rust has its own fork?
-
jdm
-
pietroalbini
jdm, yep
-
pietroalbini
jdm, it shouldn't have diverged *too* much afaik
-
jdm
yeah, there's nothing particularly surprising there
-
Manishearth
jdm: btw, blair tells me that webvr is going to go away completely
-
jdm
Manishearth: oh, I thought we had talked about that
-
jdm
I mean, I suspect blair has some unrealistic expectations
-
Manishearth
oh
-
jdm
but certainly nobody is working on the webvr standard
-
Manishearth
well
-
jdm
and google really wants to remove its implementation
-
Manishearth
I asked for what the cross interactions between the two specs are
-
jdm
but keeps putting that off because of existing content
-
Manishearth
and blair said there aren't any bc webxr replaces webvr
-
jdm
that is the goal of webxr, yes
-
Manishearth
when it comes to cross interactions the only way they won't matter is if webvr doesn't exist at all
-
Manishearth
bc otherwise they can interfere
-
jdm
theoretically, yeah
-
Manishearth
for my current impl I'm reusing stuff and ignoring that, but eventually we have some choices to make
-
» jdm -> sushi
-
KiChjang
that sounds like some big news in the standards world
-
KiChjang
then again i'd rather have standards die quick and fast
-
KiChjang
lest we get stuck in backwards-compat hell
-
ajeffrey
"💣 Failed to start rebuilding: Unknown error"
-
ajeffrey
-
crowbot
PR #22491: Add basic keyboard support for magicleap -
servo/servo #22491
-
ajeffrey
(followed by a retry)
-
ghservo
[servo] Manishearth opened issue #22526: Add DOM wrappers for dealing with typed arrays
git.io/fhfNb
-
ajeffrey
standups: Off for winter break.
-
crowbot
Status submitted successfully.