Closed Bug 1504940 Opened 6 years ago Closed 6 years ago

Remove deprecated "acceptConnections" and "singleTap" command end-points.

Categories

(Remote Protocol :: Marionette, enhancement, P2)

enhancement

Tracking

(firefox64 fixed, firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox64 --- fixed
firefox65 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

Attachments

(1 file, 1 obsolete file)

My patch on bug 1433873 introduced a new usage of a deprecated end-point, which shouldn't be used anymore.

We should get this fixed and uplifted, so that the deprecated end-point can be removed.
The patch on bug 1433873 inappropriately used the old end-point for
allowing Marionette to accept new connections again. By updating it
to the new end-point, the old one can completely be removed.
Actually the first patch on this bug is not necessary anymore given that a patch on bug 1504478 already fixed it. I requested the uplift via bug 1504478 comment 9.
Depends on: 1504478
Attachment #9022855 - Attachment is obsolete: true
With uplifting the patch on the other bug, we can also remove "singleTap" right away because it was added for Firefox 60 via bug 1451727.
Summary: Update command end-point "acceptConnections" to "Marionette:AcceptConnections" → Remove deprecated "acceptConnections" and "singleTap" command end-points.
Attachment #9022856 - Attachment description: Bug 1504940 - [marionette] Remove deprecated "acceptConnections" command end-point. r?ato → Bug 1504940 - [marionette] Remove deprecated "acceptConnections" and "singleTap" commands. r?ato
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/faf94c665ff0
[marionette] Remove deprecated "acceptConnections" and "singleTap" commands. r=ato
https://hg.mozilla.org/mozilla-central/rev/faf94c665ff0
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Whiteboard: [checkin-needed-beta]
Whiteboard: [checkin-needed-beta]
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: