Closed Bug 1622277 Opened 6 years ago Closed 5 years ago

Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_popup_blocker_identity_block.js

Categories

(Firefox :: General, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 77
Tracking Status
firefox77 --- fixed

People

(Reporter: prathiksha, Assigned: ratnabalidutta26, Mentored)

Details

Attachments

(1 file)

This is a good first bug for newcomers to Firefox development.

BrowserTestUtils.waitForCondition in browser_popup_blocker_identity_block.js[0] can be replaced by TestUtils.waitForCondition.

You can run this test with the ./mach test command:
./mach test browser/base/content/test/popups/browser_popup_blocker_identity_block.js

Please leave a comment if you would like to be assigned to this bug. Feel free to contact us via Matrix in the #osfcep channel or leave a comment here if you're stuck.

[0] https://searchfox.org/mozilla-central/source/browser/base/content/test/popups/browser_popup_blocker_identity_block.js

I'd love to pick up this ticket!

Assignee: nobody → andriacohibe
Status: NEW → ASSIGNED

Hi, are you still working on this bug? Do you need any help? :)

Flags: needinfo?(andriacohibe)
Assignee: andriacohibe → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(andriacohibe)
Assignee: nobody → ratnabalidutta26
Status: NEW → ASSIGNED
Pushed by prathikshaprasadsuman@gmail.com: https://hg.mozilla.org/integration/autoland/rev/fd542e917c85 Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_popup_blocker_identity_block.js r=prathiksha
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: