Opened 6 years ago

Closed 6 years ago

#313 closed defect (duplicate)

Renable WeakMaps

Reported by: tschuster Assignee:
Priority: Unknown Milestone:
Module: Unknown Keywords:
Cc: Blocked By:
Blocking: Platform:
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):

Description

I found this while fixing something else.
https://hg.adblockplus.org/adblockplus/file/128ab3f46c77/lib/requestNotifier.js#l34
"This branch can be enabled again once both bug 673468 and bug 819131 are fixed .."

https://bugzilla.mozilla.org/show_bug.cgi?id=673468 was fixed in Firefox 13
https://bugzilla.mozilla.org/show_bug.cgi?id=819131 was fixed in 21

So we should be able to use it.

Change History (2)

comment:1 Changed 6 years ago by saroyanm

Actually we can't yet change to WeakMaps there is another issue that we are waiting to be fixed:
https://bugzilla.mozilla.org/show_bug.cgi?id=982561

comment:2 Changed 6 years ago by saroyanm

  • Resolution set to duplicate
  • Status changed from new to closed

And also there is already a ticket for this issue, so I'm closing this one:
https://issues.adblockplus.org/ticket/47

Note: See TracTickets for help on using tickets.