Bug 97750 - merge Skipped file entries into TestExpectations, remove Skipped files
Summary: merge Skipped file entries into TestExpectations, remove Skipped files
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
Depends on: 97751
Blocks:
  Show dependency tree / graph
 
Reported: 2012-09-26 19:04 PDT by Dirk Pranke
Modified: 2012-09-27 12:23 PDT (History)
17 users (show)

See Also:

Attachments
Patch (1.58 MB, patch)
2012-09-26 19:06 PDT, Dirk Pranke 		ojan: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2012-09-26 19:04:30 PDT 
merge Skipped file entries into TestExpectations, remove Skipped files
Comment 1 Dirk Pranke 2012-09-26 19:06:15 PDT 
Created attachment 165915 [details]
Patch
Comment 2 Dirk Pranke 2012-09-26 19:07:34 PDT 
Please copy anyone else who you think might be interested in this change ...
Comment 3 Dirk Pranke 2012-09-26 19:08:31 PDT 
There should be no substantive changes in the entries in the TestExpectations files (so I would expect new-run-webkit-tests based test runs to run the same before and after this change). 

I would expect that if someone still uses old-run-webkit-tests, a few more tests might get skipped.
Comment 4 Dirk Pranke 2012-09-26 19:24:19 PDT 
We will probably want to land the patch in bug 97751 first (it makes specifying BUG modifiers optional, and so we won't get N million warnings from the style checker about lines missing them for non-chromium ports).
Comment 5 Ojan Vafai 2012-09-26 21:34:02 PDT 
Comment on attachment 165915 [details]
Patch

Yay! Might want to give a last heads up on webkit-dev and #webkit a couple hours before landing this.
Comment 6 Dirk Pranke 2012-09-27 12:23:52 PDT 
Committed r129788: <http://trac.webkit.org/changeset/129788>