1569466 - WebDriver's ErrorStatus should implement Deserialize

Status: RESOLVED FIXED

(Testing :: geckodriver, enhancement)

Description

[George Roman]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:67.0) Gecko/20100101 Firefox/67.0

Steps to reproduce:

Currently, ErrorStatus (https://searchfox.org/mozilla-central/source/testing/webdriver/src/error.rs#12) doesn't implement Deserialize.

Expected results:

It would be nice to have it implemented as it would be useful in certain scenarios. For example, Servo uses https://github.com/servo/ipc-channel for communication between the browser and the WebDriver server, which uses serde for serialization and deserialization. At the moment, we don't have a good way to distinguish between the different errors that might happen in the browser side since we can't pass an ErrorStatus back to the WebDriver server. Having Deserialize implemented would make this possible.

Comment 1

[George Roman]

Attachment: Bug 1569466 - Implement Deserialize for ErrorStatus. r=ato

Implement Deserialize for ErrorStatus

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/827775bba76d
Implement Deserialize for ErrorStatus. r=ato

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/827775bba76d