787802 - Port |Bug 762230 - Add-ons not listed when resetting Sync|

Status: RESOLVED FIXED

(SeaMonkey :: Sync UI, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch [Checkin: Comment 5]

From the base bug:
"When you go to reset Sync and replace all your data, the UI gives a list of things that will be deleted. It doesn't show add-ons because we forgot to implement it when add-on sync landed."

Note: Porting bug 787306 will be done together with bug 785225 after they land.

Comment 1

[[e-mail]]

Comment on attachment 657678 [details] [diff] [review]
patch [Checkin: Comment 5]

>+            if (i)
[I wonder what this does...]

Comment 2

[Philip Chee]

> [I wonder what this does...]

> +          for each (let i in ids) {
> +            if (i)
According to jcranmer for each is an e4xism and is deprecated.

Comment 3

[Jens Hatlak (:InvisibleSmiley)]

(In reply to Philip Chee from comment #2)
> According to jcranmer for each is an e4xism and is deprecated.

I'm not up-to-date there. If you tell me what to do instead I'll include it upon pushing, otherwise it'll have to go into a follow-up.

Comment 4

[Philip Chee]

> I'm not up-to-date there. If you tell me what to do instead I'll include it upon
> pushing, otherwise it'll have to go into a follow-up.
Not important. Let's see how the places team fixes that then we can copy it.

Comment 5

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 657678 [details] [diff] [review]
patch [Checkin: Comment 5]

http://hg.mozilla.org/comm-central/rev/42d9d1e6f548