RESOLVED FIXED Bug 143391
Reduce the dependency on Flash in regression tests 
https://bugs.webkit.org/show_bug.cgi?id=143391
Summary Reduce the dependency on Flash in regression tests
Alexey Proskuryakov
Reported 2015-04-03 16:19:28 PDT
Flash is not necessarily installed on all machines, and when it is, it can be an old version that is blocked. All these differences can cause unexpected failures, depending on what you have installed. We have a number of tests that use Flash, but somehow pass even when it is not installed. They must be not very good tests, given that they don't detect this fatal condition! I'm going to clean up as much as I can, and to make WebKitTestRunner prevent attempts at using Flash in new tests.
Attachments
patch for EWS (25.30 KB, patch)
2015-04-03 16:31 PDT, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
patch (25.30 KB, patch)
2015-04-03 16:56 PDT, Alexey Proskuryakov 		andersca: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2015-04-03 16:31:56 PDT
Created attachment 250104 [details] patch for EWS Chances are that I got at least a few cases wrong... This also marks all remaining plugins/snapshotting tests as flaky, because they don't really test snapshotting at all, and just cause trouble. We would only be interested in crashes on these tests.
Alexey Proskuryakov
Comment 2 2015-04-03 16:56:23 PDT
Created attachment 250108 [details] patch
Alexey Proskuryakov
Comment 3 2015-04-03 18:10:15 PDT
Comment on attachment 250108 [details] patch The patch fails to be applied (line endings?) But tests passed locally.
Anders Carlsson
Comment 4 2015-04-06 08:21:20 PDT
Comment on attachment 250108 [details] patch Can you remove the .swf files?
Alexey Proskuryakov
Comment 5 2015-04-06 09:32:30 PDT
Committed <http://trac.webkit.org/r182428>. > Can you remove the .swf files? Not yet - they are still used in the 8 remaining tests that are skipped on WK2, but still run on WK1. We need to associate TestNetscapePlugin with an extension, and to make it load a resource to convert these.
Note You need to log in before you can comment on or make changes to this bug.