Closed Bug 1264904 Opened 10 years ago Closed 3 years ago

ENUMERATE_WEAKARRAY should use a copied array

Categories

(Toolkit :: Places, defect, P3)

Product:
Toolkit
Component:
Places
Type:
defect

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
Tracking Status
firefox48 --- affected

People

(Reporter: mak, Unassigned)

Details

We currently don't use a copied array, that means the array can change during a notification. Since now we also expose a getObserver() API to js that exposes a copy (obviously), it makes sense to be consistent and use a copy also in ENUMERATE_WEAKARRAY.
Severity: normal → S3

I think this doesn't matter anymore with Bug 1733045

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.