Closed Bug 1789357 Opened 1 year ago Closed 1 year ago

Make AppInfo a shared class for all Remote Protocols

Categories

(Remote Protocol :: Agent, task, P1)

Product:
Remote Protocol
Component:
Agent
Type:
task
Points:
1

Tracking

(firefox106 fixed)

Status:
RESOLVED FIXED
Milestone:
106 Branch
Tracking Flags:
Tracking Status
firefox106 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Whiteboard: [webdriver:m4])

Attachments

(1 file)

Bug 1789357 - [remote] Make AppInfo a shared class for remote protocols.
48 bytes, text/x-phabricator-request
Details | Review

Right now the AppInfo class is Marionette specific but it would be great to have the class available for all the Remote Protocols.

It would help with bug 1506782 to make some checks easier.

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Points: --- → 1
Priority: -- → P1
Whiteboard: [webdriver:m4]
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0884729fca31
[remote] Make AppInfo a shared class for remote protocols. r=webdriver-reviewers,jdescottes

Backed out 2 changesets (Bug 1789357, Bug 1506782) for causing xpcshell failures on test_Capabilities.js.
Backout link
Push with failures <--> X1
Failure Log
Also Wd1 log

Flags: needinfo?(hskupin)

There is a wrong import which tries to load the AppInfo.jsm from "chrome://remote/content/remote/AppInfo.jsm".

Going to push again once it's fixed and verified locally.

Flags: needinfo?(hskupin)
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c6a1342d9065
[remote] Make AppInfo a shared class for remote protocols. r=webdriver-reviewers,jdescottes

https://hg.mozilla.org/mozilla-central/rev/c6a1342d9065

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox106: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
You need to log in before you can comment on or make changes to this bug.