Open Bug 1507251 Opened 3 years ago Updated 2 years ago

Fix test_abortPayment.html and test_canMakePayment.html to not leak in the tab process


(Core :: DOM: Web Payments, defect, P2)





(Reporter: MattN, Unassigned)



(Whiteboard: [webpayments-reserve])

For bug 1435880 I temporarily disabled the two tests in debug builds as it uncovered a leak in them. Since the leak is in the tab process it didn't seem like the UI changes in bug 1435880 caused the leak. Eden is already investigating it.

> TEST-UNEXPECTED-FAIL | leakcheck | tab process: 5040 bytes leaked (CondVar, Mutex, SchedulerEventTarget, TabGroup, ThrottledEventQueue, ...)[1]

Switching to a testing UI service also hit an assertion at which may be incorrect but I'm not sure.

Flags: qe-verify-
Assignee: MattN+bmo → echuang
Priority: P2 → P1

Is this bug still a P1?

Flags: needinfo?(echuang)

I don't think it should be P1. According to my memory, it only leaks when turning Payment UI on. Set P2 or P3 is much more reasonable.

Flags: needinfo?(echuang)
Priority: P1 → P2
Assignee: echuang → nobody
You need to log in before you can comment on or make changes to this bug.