2019

July

Monday 1st

Fixed deserialisation of error types in the Marionette Rust library prototype, improved test framework so we don’t do string comparison which vastly improves compile time by dropping regex and lazy_static.

Prepared the remote protocol meeting for tomorrow.

Investigated Puppeteer examples and commented on bug 1562638.

Tuesday 2nd

More meeting prep.

Reviewed bug 1560301.

Reviewed WPT PR 17563.

Commented on WebDriver issue 1429.

Commented on bug 1562740.

Commented on bug 1540655.

Discussion about internal inter-domain IPC with ochameau and jdescottes.

Rebased bug 1540655 and tried to figure out how to get Node.js on the image.

Closed bug 1548289.

Provided needinfo on bug 1562043.

Commented on bug 1544393.

Filed bug 1562920.

Reviewed bug 1562935.

Did some work on a patch for bug 1553756, to let Runtime.evaluate run in the current frame's execution context by default.

Wednesday 3rd

Reviewed bug 1493907.

Consulted on Nupur’s Rust library project for Marionette.

Work on bug 1540655. Found that Python was confusing TaskCluster and Puppeteer when it tried calling subprocess.Popen.communicate(). This is worth bearing in mind in the future.

Continued work on bug 1553756 to make it possible to evaluate arbitrary scripts without providing an execution context ID.

1:1 with dburns.

Reviewed bug 1493907.

Filed git-cinnabar issue 218.

1:1 with Nupur.

Submitted a naïve patch for bug 1553756 to request some feedback, especially on testing.

Thurday 4th

Reviewed bug 1563040.

Attended remote protocol project handover meeting with dburns and sole.

Rebased and submitted new try runs with increasingly more debug output for bug 1540655. I am more and more led to suspect this is an issue of stdio redirection/piping. For some reason we are getting the *** You are running in headless mode. message, which in fact is the only message in Firefox that gets printed to stderr, but nothing from stdout.

Some minor work on refining the extraction of parameter deserialisation for the Marionette Rust library project.

Did some meeting preparation for next week’s remote protocol meeting.

Filed bug 1563526 to impement Page.reload’s scriptToEvaluateOnLoad parameter.

Field bug 1563533 for introducing a shared document inlining function for the remote agent’s browser chrome tests.

Did some work on bug 1549708 in implementing Page.reload’s ignoreCache parameter. The hardest part, it turns out, is writing the tests correctly for this feature.

Email conversation about Outreachy. Brainstorming for projects, &c.

Rebased bug 1543115. Still requires some more local changes to complete.

Followed up with a try run for bug 1553756.

Commented on bug 1543115 explaining that the bug’s priority just increased because even the workaround patch changing Puppeteer from listening on stderr to stdout doesn’t work when you are dealing with shippable builds. browser.dom.window.dump.enabled is implied for debug builds, which is why my work on bug 1543115 took so long to figure out!

Reviewed bug 1563621.

Friday 5th

Filed dino-park issue 134 about making tags searchable.

Filed bug 1563714 about a build problem after upgrading to rustc 1.36.0.

Filed bug 1563746 for implementing Page.captureScreenshot.

Followed up on code review feedback for bug 1553756.

Helped answer some questions about serde for Nupur. Experimented with a solution for automatically deserialising both the command name and the parameters.

Sunday 7th

Spent some time thinking about the data structure of the Marionette Rust library.

Monday 8th

Prototyped the params-associated-with-command alternative to the Marionette Rust library.

1:1 with Nupur.

Commented on geckodriver issue 1581.

Commented on and closed bug 1563714. Turns out CARGO_BUILD_PIPELINING=true is incompatible with rustc 1.36.0.

Reviewed bug 1562043 which fixes the trace logging of packets in geckodriver following the move to use the warp request handler.

Followed up on code review issues with bug 1540655. Based on review feedback from ahal, I discovered the new test flavour likely caused all the other test harnesses to break… The good news is that the Puppeteer tests are running successfully, but we’re likely missing to populate some data that it looks for. More investigation is needed.

Reviewed bug 1563685.

Tuesday 9th

Reviewed bug 1560836, which lands the serialisation framework for the new Marionette Rust library by Nupur.

Filed bug 1564416 and bug 1564417 as a follow-up bugs for Nupur.

Commented on WPT PR 17679 about Chrome apparently appending an additional slash to directories when served over the file:// protocol.

Wednesday 10th

Reviewed WPT PR 17679, after clarifying with Ms2ger that file:// is not standardised.

Reviewed bug 1544458.

Reviewed bug 1563687.

Thursday 11th

Reviewed WPT PR 17772.

Reviewed WPT PR 17769.

Closed geckodriver issue 1583.

Reviewed bug 1562740, which tries to test the raciness we experienced with DOM propagation of keyboard input events.

Filed bug 1565183 about the message manager disappearing in favour of JSWindowActor.

Commented on D37368.

Reviewed bug 1564416, which hooks the new Marionette Rust library up to geckodriver. This is major progress!

Filed bug 1565210.

Reviewed bug 1563692, which offers a significant refactoring to the remote agent with regarads to how requests and responses are handled.

Commented on bug 1565166.

Commented on bug 1565164.

Commented on bug 1564360.

Commented on bug 1564832.

Wrote a proposal for a Summer internship project for 2020.

Commented on WPT PR 17769.

Reviewed bug 1565359.

Friday 12th

Continued working on the Summer internship proposal.

Commented on WPT PR 17795.

Attended Outreachy chit-chat meeting.

Reviewed WPT 17805.

Monday 15th

1:1 with Nupur.

Reviewed WPT PR 17824.

Reviewed WPT PR 17813.

Solved some computer setup related problems. Sigh.

Reviewed bug 1563689.

Reviewed bug 1566008.

Filed bug 1566107 and commented on bug 1462896.

Finished drafting the Summer intern proposal.