Closed Bug 1285556 Opened 4 years ago Closed 3 years ago

Add an isWebExtension getter to the AddonWrapper

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla50
Iteration:
50.4 - Aug 1
Tracking Status
firefox50 --- fixed

People

(Reporter: rpl, Assigned: rpl)

References

Details

(Whiteboard: triaged)

Attachments

(1 file)

This small change will help other components to recognize when an add-on is a WebExtension, which can be used to treat them differently (e.g. in the remote debugging actors, in the toolbox window or in the about:debugging page).

This issue is part of the proposed changes to provide an Addon Debugger with all the features of the Browser Toolbox to the WebExtensions add-ons, but, unlike the Browser Toolbox, all the sources and frames visible are going to be filtered by addon id.
Blocks: 1285557
Comment on attachment 8769202 [details]
Bug 1285556 - Add isWebExtension getter to the Addon wrapper.

https://reviewboard.mozilla.org/r/63164/#review60036
Attachment #8769202 - Flags: review?(aswan) → review+
Assignee: nobody → lgreco
Iteration: --- → 50.3 - Jul 18
Whiteboard: triaged
Blocks: 1284711
Comment on attachment 8769202 [details]
Bug 1285556 - Add isWebExtension getter to the Addon wrapper.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/63164/diff/1-2/
(In reply to Luca Greco [:rpl] from comment #3)
> Comment on attachment 8769202 [details]
> Bug 1285556 - Add isWebExtension getter to the Addon wrapper.
> 
> Review request updated; see interdiff:
> https://reviewboard.mozilla.org/r/63164/diff/1-2/

Patch rebased on a recent fx-team tip.
Status: NEW → ASSIGNED
Iteration: 50.3 - Jul 18 → 50.4 - Aug 1
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a9251411581
Add isWebExtension getter to the Addon wrapper. r=aswan
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8a9251411581
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.