Closed Bug 1497226 Opened 3 years ago Closed 3 years ago

Rename WebExtensionTargetActor.id and AddonTargetActor.id to addonId

Categories

(DevTools :: Framework, enhancement)

enhancement
Not set
normal

Tracking

(firefox64 fixed)

RESOLVED FIXED
Firefox 64
Tracking Status
firefox64 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file)

Followup to bug 1492265.

Using "id" looks very generic and might easily collide with similar field defined in the parent class (BrowsingContextTargetActor).

So it would be better to renamed this attribute to "addonId".
Summary: Rename WebExtensionTargetActor.id to addonId → Rename WebExtensionTargetActor.id and AddonTargetActor.id to addonId
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/80a1352b7cd9
Rename WebExtensionActor/WebExtensionTargetActor/AddonTargetActor.id to addonId. r=jdescottes
https://hg.mozilla.org/mozilla-central/rev/80a1352b7cd9
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Assignee: nobody → poirot.alex
You need to log in before you can comment on or make changes to this bug.