Bug 88755

Summary: Use testRunner in accessibility tests instead of layoutTestController
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: New BugsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aboxhall, cfleizach, darin, dmazzoni, eric.carlson, eric, feature-media-reviews, mitz, morrita, ojan, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 88210    
Bug Blocks:    
Attachments:
Description Flags
Patch tony: review+

Ryosuke Niwa
Reported 2012-06-10 22:58:37 PDT
Use testRunner in accessibility tests instead of layoutTestController
Attachments
Patch (349.25 KB, patch)
2012-06-10 23:01 PDT, Ryosuke Niwa 		tony: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2012-06-10 23:01:55 PDT
Created attachment 146788 [details] Patch
Ryosuke Niwa
Comment 2 2012-06-10 23:16:12 PDT
I just ran the following shell command on my Mac: find LayoutTests/accessibility LayoutTests/platform/*/accessibility -type f -not -iwholename "*.svn*" | xargs sed -i -e "s/layoutTestController/testRunner/g"
Tony Chang
Comment 3 2012-06-11 10:05:23 PDT
Why are there binary file changes?
Ryosuke Niwa
Comment 4 2012-06-11 10:08:20 PDT
(In reply to comment #3) > Why are there binary file changes? Those changes should be reverted when I land. I should have excluded .png files :(
Tony Chang
Comment 5 2012-06-11 11:05:07 PDT
Comment on attachment 146788 [details] Patch OK. A swf file also snuck in. Please exclude when landing.
Ryosuke Niwa
Comment 6 2012-06-12 13:37:23 PDT
Committed r120111: <http://trac.webkit.org/changeset/120111>
Note You need to log in before you can comment on or make changes to this bug.