Intermittent browser_pluginnotification.js | This test exceeded the timeout threshold. It should be rewritten or split up.

RESOLVED FIXED in Firefox 29

Status

()

P3
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: philor, Assigned: gfritzsche)

Tracking

({intermittent-failure})

Trunk
mozilla29
x86
Windows 8
intermittent-failure
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox27 unaffected, firefox28 unaffected, firefox29 fixed, firefox-esr24 unaffected)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=31993351&tree=Mozilla-Central
WINNT 6.2 mozilla-central debug test mochitest-browser-chrome on 2013-12-14 20:35:32 PST for push c049cb230d77
slave: t-w864-ix-093

20:45:49     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/base/content/test/general/browser_pluginnotification.js | Test 27: There should be a plugin notification
20:45:49     INFO -  [3808] WARNING: NS_ENSURE_TRUE(mMutable) failed: file c:/builds/moz2_slave/m-cen-w32-d-000000000000000000/build/netwerk/base/src/nsSimpleURI.cpp, line 265
20:45:49  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_pluginnotification.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort.
20:45:49     INFO -  TEST-INFO | MEMORY STAT vsize after test: 892813312
20:45:49     INFO -  TEST-INFO | MEMORY STAT vsizeMaxContiguous after test: 1893662720
20:45:49     INFO -  TEST-INFO | MEMORY STAT residentFast after test: 380284928
20:45:49     INFO -  INFO TEST-END | chrome://mochitests/content/browser/browser/base/content/test/general/browser_pluginnotification.js | finished in 91160ms
Comment hidden (Treeherder Robot)
(Assignee)

Comment 2

5 years ago
No single step in the logs is taking particularly long, i think that bug 932832 etc just tipped it over the edge.
We should break a few of those tests out and definitely not add anything new there.
Priority: -- → P3
(Assignee)

Comment 3

5 years ago
Created attachment 8350606 [details] [diff] [review]
Move click-to-play notification bar tests out

Fine on try: https://tbpl.mozilla.org/?tree=Try&rev=0eba696e4789
Assignee: nobody → georg.fritzsche
Attachment #8350606 - Flags: review?(jaws)
https://hg.mozilla.org/mozilla-central/rev/a41fa86a7850
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
status-firefox27: --- → unaffected
status-firefox28: --- → unaffected
status-firefox29: --- → fixed
status-firefox-esr24: --- → unaffected
You need to log in before you can comment on or make changes to this bug.