906703 - Intermittent browser_pluginnotification.js | Test N, Should have displayed the missing plugin notification

Status: RESOLVED FIXED

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

Description

[Georg Fritzsche [:gfritzsche]]

Follow-up to bug 896965.

RyanVM
https://tbpl.mozilla.org/php/getParsedLog.php?id=26650580&tree=Mozilla-Beta
WINNT 6.2 mozilla-beta pgo test mochitest-browser-chrome on 2013-08-16 12:45:01
revision: f676984e6e04
slave: t-w864-ix-096

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test 1, Should have displayed the missing plugin notification
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test 1, Should be a missing plugin list
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | uncaught exception - TypeError: gTestBrowser.missingPlugins is null at chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js:117
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Found a tab after previous test timed out: chrome://mochitests/content/browser/browser/base/content/test/plugin_unknown.html
TypeError: gControllerSources.getCache is not a function: testStateBeforeReload@chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_sources-cache.js:117

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Add missing forcing of layout flush

Jaws, let me know if i'm flooding you with too many CTP-related review requests.

This adds forcing the layout flush before the first check on the plugins-not-found notification. In bug 896965 i missed that this notification is also triggered by "PluginBindingAttached" and hence also requires forcing a layout flush.

Comment 2

[Georg Fritzsche [:gfritzsche]]

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

Comment 3

[Ed Morley [:emorley]]

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

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/4202492ea7d4
https://hg.mozilla.org/releases/mozilla-beta/rev/bdc6c32b56cb

Comment 5

[Legacy TBPL/Treeherder Robot]

msucan
https://tbpl.mozilla.org/php/getParsedLog.php?id=27234528&tree=Try
win64_vm try opt test mochitest-browser-chrome on 2013-08-30 12:03:29
revision: 820531ff781f
slave: tst-w64-ec2-001

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_CTP_data_urls.js | Test 2b, Should have a click-to-play notification
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_CTP_data_urls.js | uncaught exception - TypeError: notification is null at chrome://mochitests/content/browser/browser/base/content/test/browser_CTP_data_urls.js:165
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_CTP_data_urls.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_CTP_data_urls.js | Found a tab after previous test timed out: data:text/html,<embed%20id="test1"%20style="width:%20200px;%20height:%20200px"%20type="application/x-test"/><embed%20id="test2"%20style="width:%20200px;%20height:%20200px"%20type="application/x-second-test"/>
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug752516.js | uncaught exception - TypeError: PopupNotifications.getNotification(...) is null at chrome://mochitests/content/browser/browser/base/content/test/browser_bug752516.js:43
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug752516.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug787619.js | uncaught exception - TypeError: PopupNotifications.getNotification(...) is null at chrome://mochitests/content/browser/browser/base/content/test/browser_bug787619.js:38
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug787619.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug787619.js | Found a tab after previous test timed out: http://127.0.0.1:8888/browser/browser/base/content/test/plugin_bug787619.html
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_bug818118.js | should have a click-to-play notification
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test 6, Should have displayed the missing plugin notification
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test 6, Should be a missing plugin list
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | uncaught exception - TypeError: gTestBrowser.missingPlugins is null at chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js:200
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginnotification.js | Found a tab after previous test timed out: chrome://mochitests/content/browser/browser/base/content/test/plugin_both.html
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginplaypreview.js | Test 2, Plugin should be activated
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginplaypreview.js | Test 3, Plugin should be activated
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginplaypreview.js | Test 4a, Waited too long for plugin to stop play preview
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginplaypreview2.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_pluginplaypreview2.js | Found a tab after previous test timed out: chrome://mochitests/content/browser/browser/base/content/test/plugin_test.html
Only displaying first 20 of 67 failures -
.