1445267 - Replace promiseTopicObserved with TestUtils.topicObserved in browser_testOpenNewRemoteTabsFromNonRemoteBrowsers.js

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P5)

Description

[Johann Hofmann [:johannh]]

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

promiseTopicObserved in the browser_testOpenNewRemoteTabsFromNonRemoteBrowsers.js test file can be replaced by the TestUtils.topicObserved[0] utility function.

We need to replace this occurrence: https://searchfox.org/mozilla-central/rev/99df6dad057b94c2ac4d8b12f7b0b3a7fdf25d04/browser/base/content/test/general/browser_testOpenNewRemoteTabsFromNonRemoteBrowsers.js#107

It seems to me that this is also the last time we use the promiseTopicObserved function in browser/base/content/test/general, which means we can also remove its definition here: https://searchfox.org/mozilla-central/rev/99df6dad057b94c2ac4d8b12f7b0b3a7fdf25d04/browser/base/content/test/general/head.js#540


For instructions on how to get your local build of Firefox up and running and submit your patch, see https://developer.mozilla.org/en-US/docs/Introduction.

You can run this test with the ./mach mochitest command:

./mach mochitest browser/base/content/test/general/browser_testOpenNewRemoteTabsFromNonRemoteBrowsers.js

Please leave a comment if you would like to be assigned to this bug and feel free to ask questions here or via IRC if you're stuck.

[0] https://searchfox.org/mozilla-central/rev/588d8120aa11738657da93e09a03378bcd1ba8ec/testing/modules/TestUtils.jsm#56

Comment 1

[utsha.sinha1510]

I would like to work on this issue. Is it available?

Comment 2

[kev.ftw595]

Can I work on the bug?

Comment 3

[Johann Hofmann [:johannh]]

utsha.sinha1510 was first :)

(In reply to kev.ftw595 from comment #2)
> Can I work on the bug?

How about bug 1447941 instead? :)

Comment 4

[Johann Hofmann [:johannh]]

Hi, are you still working on this? Is there anything I can help you with?

Comment 5

[utsha.sinha1510]

Go ahead. Take over. I'm kind of new and it will take time to get acquainted.

Comment 6

[Johann Hofmann [:johannh]]

Thanks!

Comment 7

[ryanro1997]

Hello, I would like to be assigned this bug.

Comment 8

[Johann Hofmann [:johannh]]

Of course, let me know if you need any help.

Thank you!

Comment 9

[ryanro1997]

I was wondering if there was a way to fix:
FAIL test left unexpected property on window: TestUtils -
which happens after this info message:
Leaving test bound test_new_window
All other tests pass.

Thanks.

Comment 10

[ryanro1997]

(In reply to ryanro1997 from comment #9)
> I was wondering if there was a way to fix:
> FAIL test left unexpected property on window: TestUtils -
> which happens after this info message:
> Leaving test bound test_new_window
> All other tests pass.
> 
> Thanks.

It seems it was due to me adding the line:
ChromeUtils.import("resource://testing-common/TestUtils.jsm");
Which was not needed.

Comment 11

[ryanro1997]

Attachment: Bug 1445267 - Removed deprecated definition promiseTopicObserved and replaced call with TestUtils.topicObserved, r=jhofmann.patch

Comment 12

[Johann Hofmann [:johannh]]

Comment on attachment 8965286 [details] [diff] [review]
Bug 1445267 - Removed deprecated definition promiseTopicObserved and replaced call with TestUtils.topicObserved, r=jhofmann.patch

Review of attachment 8965286 [details] [diff] [review]:
-----------------------------------------------------------------

Ah, excellent! This looks good. Let's give it a quick run on our Try server (https://wiki.mozilla.org/ReleaseEngineering/TryServer), to verify that all tests are still passing. I'll also set the checkin-needed flag so that someone will land the patch when try is green.

Thanks!

Comment 13

[Johann Hofmann [:johannh]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4350580a669fc4275f99f957c4697c268ef391bd

Comment 14

[Johann Hofmann [:johannh]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/af78c390b5d2b9d114093836b9c811249211ceda
Bug 1445267 - Removed deprecated definition promiseTopicObserved and replaced call with TestUtils.topicObserved. r=johannh

Comment 15

[Andreea Pavel [:apavel]]

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