Closed
Bug 1287233
Opened 8 years ago
Closed 6 years ago
browser.test / chrome.test API should not be public
Categories
(WebExtensions :: Untriaged, defect, P5)
WebExtensions
Untriaged
Tracking
(firefox50 affected)
RESOLVED
INACTIVE
Tracking | Status | |
---|---|---|
firefox50 | --- | affected |
People
(Reporter: robwu, Unassigned)
References
(Depends on 1 open bug)
Details
(Whiteboard: triaged)
Currently chrome.test only implements harmless assertion APIs, but that may change in the future. If it becomes anywhere as powerful as Chrome's chrome.test API, then we have a problem.
Bill said that it's doable to put the API behind a preference and run tests with that preference.
Comment 1•8 years ago
|
||
I don't think it's worth wasting time on this. All this API does is pass messages to listeners that the test harness adds to the extension instance. If at some point we decide that this needs to be hidden, we can just expose it only when there are listeners. But since the API is entirely a no-op in that case, I don't think it's worth the effort.
Reporter | ||
Comment 2•8 years ago
|
||
At the moment, that's indeed what the API does.
That's no guarantee for the future, however: unnecessarily exposed APIs are a potential security risk.
The risk is currently low, so I agree that this is a low-prio issue.
Updated•8 years ago
|
Priority: -- → P5
Whiteboard: triaged
Comment 3•6 years ago
|
||
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
Updated•6 years ago
|
Product: Toolkit → WebExtensions
Reporter | ||
Comment 4•5 years ago
|
||
The first step towards unpublishing the API implementation is done here: https://phabricator.services.mozilla.com/D53981
Basically, outside of tests the API bindings are still generated, but the implementation is missing.
You need to log in
before you can comment on or make changes to this bug.
Description
•