Bug 42373

Summary: Changes to enable MathML
Product: WebKit Reporter: Alex Milowski <alex>
Component: MathMLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, commit-queue, eric, martijnc, peter, sausset
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 3251    
Attachments:
Description Flags
Enable MathML Patch bdakin: review+, bdakin: commit-queue-

Alex Milowski
Reported 2010-07-15 07:31:13 PDT
Created attachment 61658 [details] Enable MathML Patch The attached patch changes the defaults so that MathML is enabled. People can turn it off with --no-mathml on build-webkit.
Attachments
Enable MathML Patch (5.56 KB, patch)
2010-07-15 07:31 PDT, Alex Milowski 		bdakin: review+
bdakin: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2010-07-27 07:17:04 PDT
Comment on attachment 61658 [details] Enable MathML Patch This is very exciting to me. I don't know enough about the MathML status to know whether we're ready, but yay!
Beth Dakin
Comment 2 2010-08-03 10:45:24 PDT
Comment on attachment 61658 [details] Enable MathML Patch I did some performance testing and didn't find any issues. I also haven't encountered any stability issues, and all the layout tests appear to pass.
Beth Dakin
Comment 3 2010-08-03 14:23:06 PDT
Comment on attachment 61658 [details] Enable MathML Patch Changing commit-queue flag to minus. I want to make sure this is committed when I am awake to help clean up any possible messes, so I am going to commit this by hand.
Eric Seidel (no email)
Comment 4 2010-08-03 14:35:05 PDT
Makes sense. Sorry for the massive cq delays the last 24 hours.
Beth Dakin
Comment 5 2010-08-03 14:36:27 PDT
(In reply to comment #4) > Makes sense. Sorry for the massive cq delays the last 24 hours. No worries :-) Committed with http://trac.webkit.org/changeset/64589
Beth Dakin
Comment 6 2010-08-03 15:12:04 PDT
Speculative build fix: http://trac.webkit.org/changeset/64592
Beth Dakin
Comment 7 2010-08-03 15:43:47 PDT
Speculative build fix part 2.
Beth Dakin
Comment 8 2010-08-03 15:44:23 PDT
(In reply to comment #7) > Speculative build fix part 2. Oops! Forgot to add the link: http://trac.webkit.org/changeset/64596
Beth Dakin
Comment 9 2010-08-03 16:39:14 PDT
Another build fix: http://trac.webkit.org/changeset/64600
Beth Dakin
Comment 10 2010-08-03 16:45:10 PDT
This too: http://trac.webkit.org/changeset/64607
Note You need to log in before you can comment on or make changes to this bug.