Philip Jägenstedt

Results 565 comments of Philip Jägenstedt

@philipwalton I wanted to submit the same change when I spotted this open PR. Can you review this?

In addition, the individual bugs should be marked as fixed: https://bugs.chromium.org/p/chromium/issues/detail?id=375693 in 86 and https://bugs.chromium.org/p/chromium/issues/detail?id=700029 in 82.

Talking about the bits of markup that are valid and not what *changes* to markup are confirming would probably help. Removing the tag in the two cases results in the...

I've done a fair bit of work on getting Safari working well with [web-platform-tests](https://github.com/web-platform-tests/wpt) (a conformance test suite for browsers with a [dashboard](https://wpt.fyi/)) and know a little bit about what...

Thanks @Bkucera, that's a great list! > get/set/clear cookies by domain It sounds like you need to get/set/clear cookies for domains other than the one currently loaded, is that right?...

@jcscottiii it looks like this has been happening with some regularity looking at https://wpt.fyi/runs?label=master&label=stable&max-count=100&product=chrome&product=firefox&product=safari Here's how I'd investigate what went wrong. I'll use the runs missing in 6808a6b as the...

Taking another pair of runs, from the previous commit: - Chrome with 47446 tests: https://wpt.fyi/results/?run_id=5728025067913216 - Edge with 46711 tests: https://wpt.fyi/results/?run_id=5758360556142592 The Chrome number is the same but the Edge...

@domenic can you check if the failure in https://community-tc.services.mozilla.com/tasks/d9IN_UANTrG29Eav08BPaA/runs/0/logs/public/logs/live.log is plausible caused by the changes?

I pushed https://github.com/web-platform-tests/wpt/pull/28759 and the infrastructure/ tests are passing there. I'll try rebasing onto the exact same commit (4e1dddf5a4bbd613747f04af48d9f150ff0a1df6) to see if it still fails here.

The errors looks a lot like the cert is broken and the tests don't load at all. @domenic how did you update the certs here?