Closed Bug 1703126 Opened 3 years ago Closed 3 years ago

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

Categories

(Firefox :: General, task, P5)

task

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: prathiksha, Assigned: theodorasabalou, Mentored)

Details

Attachments

(2 files)

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

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

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

Please leave a comment if you would like to be assigned to this bug.

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

Hello! I would like to be assigned to this bug.

I would like to contribute this is my first time contributing to open source plz help me in th way

I would like to contribute this is my first time contributing to open source plz help me in th way

Assignee: nobody → theodorasabalou
Status: NEW → ASSIGNED

Bug 1703126 - Replaced BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_geolocation_indicator.js r=prathiksha


Bug 1703126 - Replaced BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_geolocation_indicator. r=prathiksha

Attached file Link to bug commit
Pushed by prathikshaprasadsuman@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/09482c058bc1
Replaced BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_geolocation_indicator.  r=prathiksha
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.