1477476 - Drop check for malformed sessionId from Marionette

Status: RESOLVED FIXED

(Testing :: geckodriver, enhancement)

Description

[Andreas Tolfsen ❲:ato❳]

geckodriver checks if the sessionId field returned from Marionette
starts with "{" and ends with "}", which it hasn’t done for quite
some time:

> if session_id.starts_with("{") && session_id.ends_with("}") {
>     session_id = &session_id[1..session_id.len()-1];
> }

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Drop geckodriver check for malformed sessionId from Marionette. r=automatedtester

Marionette used to return a session UUID starting with "{" and
ending with "}" from the uuid-generator in Gecko.  It has not done
this for a very long time rendering this geckodriver check needless.

Comment 2

[Andreas Tolfsen ❲:ato❳]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b07d67b158a9b071f73f00bc49d47bda22429f31

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/666d7dc7a32c
Drop geckodriver check for malformed sessionId from Marionette. r=automatedtester

Comment 4

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/666d7dc7a32c