search

web-platform-tests / rfcs

web-platform-tests RFCs
75 stars 63 forks source link

RFC 74: Add testdriver.js support for WebDriver delete_all_cookies command #74

Closed miketaylr closed 3 years ago

miketaylr commented 3 years ago

PR to add support for delete_all_cookies here: https://github.com/web-platform-tests/wpt/pull/26723

annevk commented 3 years ago

WebDriver doesn't seem particularly detailed about how it serializes cookie values. That seems rather important to address first, at least before adding anything apart from "delete all", given that JSON doesn't have a native representation for byte sequences. (This concern might also apply to cookie names, not sure.)

miketaylr commented 3 years ago

at least before adding anything apart from "delete all"

This is the one bit I'm actually interested in right now, so I'm happy to reduce the scope of the RFC to land this without blocking on the rest.

annevk commented 3 years ago

Sounds good, I filed https://github.com/w3c/webdriver/issues/1562 on WebDriver for the serialization issue.

jgraham commented 3 years ago

I think this makes sense.

miketaylr commented 3 years ago

Sounds good, I filed w3c/webdriver#1562 on WebDriver for the serialization issue.

Thanks. I've updated the RFC as well.

miketaylr commented 3 years ago

It's been 7 days with no objections, shall we merge this?