Closed Bug 1690221 Opened 4 years ago Closed 4 years ago

Remove unused chrome connector and merge FirefoxConnector into Connector

Categories

(DevTools :: Netmonitor, enhancement)

enhancement

Tracking

(firefox87 fixed)

RESOLVED FIXED
87 Branch
Tracking Status
firefox87 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Network monitor is having a "chrome connector", which used to allow the netmonitor to inspector chrome requests:
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/src/connector/chrome-connector.js
But this is not longer used, nor usable since we removed launchpad support.

It would probably help getting rid of this abstraction and merge FirefoxConnector into Connector.
We could then iterate and see how to possibly merge FirefoxDataProvider into Connector and/or move part of FirefoxDataProvider to ResourceWatcher/ResourceTransformer/...

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/31609c430fd8 [devtools] Remove unused chrome connector. r=bomsy https://hg.mozilla.org/integration/autoland/rev/22706437cd15 [devtools] Merge FirefoxConnector into Connector. r=bomsy https://hg.mozilla.org/integration/autoland/rev/abfdad7b2fab [devtools] Remove unused Connector.setPreferences method. r=bomsy
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: