Bug 29349

Summary: Added check for MathML support to run-webkit-tests
Product: WebKit Reporter: Alex Milowski <alex>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 29158    
Attachments:
Description Flags
Patch supporting testing for MathML
mjs: review-
Updated patch with extra checks removed none

Alex Milowski
Reported 2009-09-17 15:26:13 PDT
Created attachment 39729 [details] Patch supporting testing for MathML Added the checks to the support libraries to check to see in MathML has been compiled. If not, then the mathml layout tests are ignored.
Attachments
Patch supporting testing for MathML (2.19 KB, patch)
2009-09-17 15:26 PDT, Alex Milowski 		mjs: review-
DetailsFormatted DiffDiff
Updated patch with extra checks removed (2.09 KB, patch)
2009-09-17 18:25 PDT, Alex Milowski 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2009-09-17 18:20:03 PDT
Comment on attachment 39729 [details] Patch supporting testing for MathML Why does this hardcode the MathML support values for Qt and Wx? That doesn't seem right...
Alex Milowski
Comment 2 2009-09-17 18:25:14 PDT
Created attachment 39743 [details] Updated patch with extra checks removed
Maciej Stachowiak
Comment 3 2009-09-17 18:43:08 PDT
Comment on attachment 39743 [details] Updated patch with extra checks removed r=me
Eric Seidel (no email)
Comment 4 2009-09-18 11:44:33 PDT
Comment on attachment 39743 [details] Updated patch with extra checks removed IIRC Alex is not a committer, adding cq+
WebKit Commit Bot
Comment 5 2009-09-18 12:06:23 PDT
Comment on attachment 39743 [details] Updated patch with extra checks removed Clearing flags on attachment: 39743 Committed r48522: <http://trac.webkit.org/changeset/48522>
WebKit Commit Bot
Comment 6 2009-09-18 12:06:26 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.