Andreas Tolfsen ❲:ato❳ Reporter
	Description • 9 years ago

We currently don't throw an error when deleteCookie can't find the cookie.

https://dxr.mozilla.org/mozilla-central/source/testing/marionette/marionette-listener.js?from=marionette-listener.js#1952

	David Burns :automatedtester
	Comment 1 • 9 years ago

This appears to do the exact same thing as WebDriver OSS[1]

[1] https://code.google.com/p/selenium/source/browse/javascript/firefox-driver/js/firefoxDriver.js#790

What error do you propose we throw?

	Andreas Tolfsen ❲:ato❳ Reporter
	Comment 2 • 9 years ago

(In reply to David Burns :automatedtester from comment #1)
> This appears to do the exact same thing as WebDriver OSS[1]
> 
> [1]
> https://code.google.com/p/selenium/source/browse/javascript/firefox-driver/
> js/firefoxDriver.js#790
> 
> What error do you propose we throw?

I don't think any of the existing error codes in Selenium are useful, and it isn't described in spec yet.  We should probably introduce a new status "cookie not found" or similar.

	Andreas Tolfsen ❲:ato❳ Reporter
	Comment 3 • 8 years ago

I’ve submitted https://github.com/w3c/webdriver/pull/303 to the specification.

	Andreas Tolfsen ❲:ato❳ Reporter
	Comment 4 • 8 years ago

I have a feeling that the count of deleted cookies is not something we can reliably track.  Implementations may also not even follow the exact steps in this context. (Marionette for example uses the Gecko cookie service in chrome space.)