Bug 42132 - Need support for a WebKit2-specific Skipped list (and initially add accessibility tests to it)
Summary: Need support for a WebKit2-specific Skipped list (and initially add accessibi...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Maciej Stachowiak
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-12 19:52 PDT by Maciej Stachowiak
Modified: 2010-07-12 20:18 PDT (History)
1 user (show)

See Also:


Attachments
Patch (5.38 KB, patch)
2010-07-12 19:58 PDT, Maciej Stachowiak
sam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Stachowiak 2010-07-12 19:52:36 PDT
Need support for a WebKit2-specific Skipped list (and initially add accessibility tests to it)
Comment 1 Maciej Stachowiak 2010-07-12 19:58:48 PDT
Created attachment 61316 [details]
Patch
Comment 2 WebKit Review Bot 2010-07-12 20:00:30 PDT
Attachment 61316 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKitTools/Scripts/old-run-webkit-tests:351:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:352:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1886:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1887:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1888:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1889:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1890:  Line contains tab character.  [whitespace/tab] [5]
WebKitTools/Scripts/old-run-webkit-tests:1921:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 8 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Sam Weinig 2010-07-12 20:01:47 PDT
Comment on attachment 61316 [details]
Patch

>  
> +if ($useWebKitTestRunner) {
> +    if (isAppleMacWebKit()) {
> +	$realPlatform = $platform;
> +	$platform = "mac-wk2";
> +    }
> +}

Looks like a case of the "tabs." There are a few others of these.



> +# THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> +#
> +
> +# <https://bugs.webkit.org/show_bug.cgi?id=42130> WebKit2 needs to support accessibility
> +# <https://bugs.webkit.org/show_bug.cgi?id=42131> WebKitTestRunner needs to support accessibility-related DRT APIs
> +accessibility
> \ No newline at end of file

Need a newline.

Other than that, r=me.
Comment 4 Maciej Stachowiak 2010-07-12 20:18:03 PDT
http://trac.webkit.org/changeset/63163