The call to Page::refreshPlugins(reloadOpenPages) is missing from refreshPlugins() implementation in PluginInfoStore.cpp .
Will upload a patch after I test on all platforms.
Created attachment 43596 [details]
Comment on attachment 43596 [details]
Rejecting patch 43596 from commit-queue.
Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11651 test cases.
plugins/reloadplugins-and-pages.html -> failed
Exiting early after 1 failures. 9585 tests run.
337.89s total testing time
9584 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
7 test cases (<1%) had stderr output
Created attachment 43603 [details]
I forgot that not all ports share the same plugins code. Added the new tests to the skip list for ports that don't use WebCore/plugins.
Comment on attachment 43603 [details]
Rejecting patch 43603 from commit-queue.
Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Kenneth Rohde Christiansen', '--force']" exit_code: 1
Last 500 characters of output:
patching file LayoutTests/plugins/reloadplugins-no-pages-expected.txt
patching file LayoutTests/plugins/reloadplugins-and-pages-expected.txt
patching file LayoutTests/plugins/reloadplugins-no-pages.html
patching file LayoutTests/plugins/reloadplugins-and-pages.html
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 1.
patching file WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
Committed 51257 <http://trac.webkit.org/changeset/51257>
Hmm, the Mac port uses a different code path indeed, but the Safari Windows build uses WebCore/plugins/npapi.cpp, which calls the refreshPlugins() function you changed. So theoretically the test should pass there. The bot output only indicated failure on Mac.