Closed Bug 1409030 Opened 2 years ago Closed 2 years ago

Remove protocol levels below 3

Categories

(Testing :: Marionette, enhancement)

57 Branch
enhancement
Not set

Tracking

(firefox58 fixed)

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Keywords: pi-marionette-client)

Attachments

(1 file)

We upgraded to the protocol level 3 a long time ago. It's no reason to longer keep this level around given that it only makes the code more complicated.

As talked with Andreas we will remove it, but leave `self.protocol` in Marionette still intact in case of future protocol changes.
Attachment #8918887 - Flags: review?(ato)
Comment on attachment 8918887 [details]
Bug 1409030 - Remove Marionette socket protocol levels below 3.

https://reviewboard.mozilla.org/r/189770/#review194918
Attachment #8918887 - Flags: review?(ato) → review+
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7f85a2865e4d
Remove Marionette socket protocol levels below 3. r=ato
https://hg.mozilla.org/mozilla-central/rev/7f85a2865e4d
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Depends on: 1426082
You need to log in before you can comment on or make changes to this bug.