1166552 - browser_newtab_reportLinkAction.js fails on timeout

Status: VERIFIED FIXED

(Firefox :: New Tab Page, defect)

Description

[maxim zhilyaev]

./mach mochitest-browser browser/base/content/test/newtab/browser_newtab_reportLinkAction.js 

hangs on MacOS, while displaying a unicorn picture "New tab got an update!".

Comment 1

[Ed Lee :Mardak]

This probably passes when all the other mochitests are run and cause the onboarding to have already been shown.

We should do the same for the update pref:

http://mxr.mozilla.org/mozilla-central/source/testing/profiles/prefs_general.js#192

Comment 2

[Ed Lee :Mardak]

Attachment: v1

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/c20e5947d8dd

Comment 4

[Carsten Book [:Tomcat]]

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

Comment 5

[Ed Lee :Mardak]

Attachment: for aurora (Mardak will land)

Comment 6

[Ed Lee :Mardak]

Attachment: for beta (Mardak will land)

Comment 7

[maxim zhilyaev]

Verified on most recent gecko-dev build. the test does not hang/fail anymore
output of git log is attached below:

>>
~/gecko-dev$ git lg
*   a9bac08 - (HEAD, origin/master, origin/HEAD, master) merge mozilla-inbound to mozilla-central a=merge (8 hours ago) <Carsten "Tomcat" Book>
|\  
| * 838880e - Bug 1163497. Part 5 - remove unused functions. r=roc. (16 hours ago) <JW Wang>
| * 5db825a - Bug 1163497. Part 4 - move output stream code from MediaDecoder::AddOutputStream() to DecodedStream::Connect(). r=roc. (16 hours ago) <JW Wa
| * 5871f0d - Bug 1163497. Part 3 - move stream connection code from MediaDecoder::RecreateDecodedStream() to DecodedStream::RecreateData(). r=roc. (16 ho
| * 4db24fa - Bug 1163497. Part 2 - Remove MediaDecoder::DestroyDecodedStream(). r=roc. (2 weeks ago) <JW Wang>
| * 6166489 - Bug 1163497. Part 1 - remove MediaDecoder::ConnectDecodedStreamToOutputStream(). r=roc. (2 weeks ago) <JW Wang>
| *   9a4f92d - Merge m-c to m-i (16 hours ago) <Phil Ringnalda>

Comment 8

[Ed Lee :Mardak]

Comment on attachment 8610029 [details] [diff] [review]
for aurora (Mardak will land)

Approval Request Comment: See bug 1140185 comment 11

Comment 9

[Ed Lee :Mardak]

http://hg.mozilla.org/releases/mozilla-aurora/rev/303f9fa801b6

Comment 10

[Ed Lee :Mardak]

Comment on attachment 8610029 [details] [diff] [review]
for aurora (Mardak will land)

approval-mozilla-aurora+ granted in bug 1140185 comment 14