1373218 - InvalidCookieDomainError not included in STATUSES map

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Andreas Tolfsen ❲:ato❳]

THe InvalidCookieDomainError type in testing/marionette/error.js is not included in the STATUSES map, causing a failure to look it up as can be seen in https://github.com/mozilla/geckodriver/issues/782.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1373218 - Add invalid cookie domain error to status lookup;

The InvalidCookieDomainError type is missing a status lookup key in
testing/marionette/error.js.  This adds "invalid cookie domain" as a
reverse lookup for the type.

Review commit: https://reviewboard.mozilla.org/r/149406/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/149406/

Comment 2

[Andreas Tolfsen ❲:ato❳]

This is a good candidate for uplift to beta (55).

Comment 3

[David Burns :automatedtester]

Comment on attachment 8877996 [details]
Bug 1373218 - Add invalid cookie domain error to status lookup;

https://reviewboard.mozilla.org/r/149406/#review153954

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6609469c2427
Add invalid cookie domain error to status lookup; r=automatedtester

Comment 5

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

https://hg.mozilla.org/mozilla-central/rev/6609469c2427

Comment 6

[Andreas Tolfsen ❲:ato❳]

Sheriffs: Requesting uplift to beta (55) as test-only.  This is considered low risk.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1c40b8e89ae4