1702452 - Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_tabSwitchPrintPreview.js

Status: RESOLVED FIXED

(Firefox :: General, task, P5)

Description

[:prathiksha]

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

BrowserTestUtils.waitForCondition in browser_tabSwitchPrintPreview.js[0] can be replaced with TestUtils.waitForCondition.

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

Feel free to get in touch with me on Matrix if you're stuck.

[0] https://searchfox.org/mozilla-central/rev/4fa18c26fa907f38d56b599571b9846af1506f3c/browser/base/content/test/tabs/browser_tabSwitchPrintPreview.js

Comment 1

[rashadmad]

Hello I am Rashad I would love this to be assigned to me as my first bug.

Comment 2

[rashadmad]

I was wondering where is the changeset ID located? I need it to push my changes to mozphab.

Comment 3

[rashadmad]

Attachment: Bug 1702452 - Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_tabSwitchPrintPreview.js. r=prathiksha

Comment 4

[rashadmad]

never mind about the changeset question, I pushed up a change to phabricator hopefully all is well if not just let me know: https://bugzilla.mozilla.org/show_bug.cgi?id=1702452 if everything is alright is it possible for me to be assigned another ticket? I dont know if you take weekends off.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ed3c488e92c8
Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_tabSwitchPrintPreview.js. r=prathiksha

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/ed3c488e92c8