702806 - Create AddonManager.getAllStartupChanges()

Status: RESOLVED INACTIVE

(Toolkit :: Add-ons Manager, defect)

Description

[Gregory Szorc [:gps]]

Attachment: Add AddonManager.getAllStartupChanges()

I'm requesting a new function on AddonManager, getAllStartupChanges().

This will simply return an object containing IDs of everything that changed during startup. e.g.

  {changed: ["id1"], installed: ["id2"}

I want this API for add-on sync and think it is better than calling getStartupChanges() for each change type because: a) less function calls b) guaranteed to iterate over every change type.

If the attachment is f+, I'll code some unit tests and submit a new patch for review. If you don't like the idea, it will make Sync code more verbose and prone to breaking, but I'll live with it.

Comment 1

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Comment on attachment 574710 [details] [diff] [review]
Add AddonManager.getAllStartupChanges()

Review of attachment 574710 [details] [diff] [review]:
-----------------------------------------------------------------

Yea, that works. Pity the design of getStartupChanges() doesn't allow for just passing in STARTUP_CHANGE_ALL or null (like the rest of the APIs do), but oh well.

Comment 2

[Gregory Szorc [:gps]]

Well, it would be trivial to add STARTUP_CHANGE_ALL to getStartupChanges(). The return type would have a different format (object instead of list), so maybe a different function is warranted. Let me know.

Comment 3

[Blair McBride [:Unfocused] (UNAVAILABLE)]

(In reply to Gregory Szorc [:gps] from comment #2)
> The return type would have a different format (object instead of list), so
> maybe a different function is warranted.

Yea, that's what I meant - it would require changing the return type, which I don't think is worth it since that would make the normal use of that function more cumbersome. So a new function seems like the way to go.

Comment 4

[Gregory Szorc [:gps]]

I may not need this API depending on what happens in bug 702819. Ideally this API should be considered separately. But since I'm the only requester, I'm going to hold off and see what happens in bug 702819 before going any further.

Comment 5

[Gregory Szorc [:gps]]

I no longer need this for add-on sync. I'll leave it up to the Add-on Manager folks to determine what to do with it.

Comment 6

[BMO Automation]

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.