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

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect, P3)

Description

[Phil Ringnalda (:philor)]

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 1

[Legacy TBPL/Treeherder Robot]

hsivonen
https://tbpl.mozilla.org/php/getParsedLog.php?id=32183688&tree=Holly
Windows XP 32-bit holly debug test mochitest-browser-chrome on 2013-12-18 13:49:24
revision: 072edc42109d
slave: t-xp32-ix-061

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.
Return code: 1

Comment 2

[Georg Fritzsche [:gfritzsche]]

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.

Comment 3

[Georg Fritzsche [:gfritzsche]]

Attachment: Move click-to-play notification bar tests out

Fine on try: https://tbpl.mozilla.org/?tree=Try&rev=0eba696e4789

Comment 4

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a41fa86a7850

Comment 5

[Ryan VanderMeulen [:RyanVM]]

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