Bug 42373 - Changes to enable MathML
: Changes to enable MathML
Status: RESOLVED FIXED
: WebKit
MathML
: 528+ (Nightly build)
: All Mac OS X 10.5
: P2 Normal
Assigned To:
:
:
:
: 3251
  Show dependency treegraph
 
Reported: 2010-07-15 07:31 PST by
Modified: 2010-08-03 16:45 PST (History)


Attachments
Enable MathML Patch (5.56 KB, patch)
2010-07-15 07:31 PST, Alex Milowski
bdakin: review+
bdakin: commit‑queue-
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-07-15 07:31:13 PST
Created an attachment (id=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.
------- Comment #1 From 2010-07-27 07:17:04 PST -------
(From update of attachment 61658 [details])
This is very exciting to me.  I don't know enough about the MathML status to know whether we're ready, but yay!
------- Comment #2 From 2010-08-03 10:45:24 PST -------
(From update of attachment 61658 [details])
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.
------- Comment #3 From 2010-08-03 14:23:06 PST -------
(From update of attachment 61658 [details])
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.
------- Comment #4 From 2010-08-03 14:35:05 PST -------
Makes sense.  Sorry for the massive cq delays the last 24 hours.
------- Comment #5 From 2010-08-03 14:36:27 PST -------
(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
------- Comment #6 From 2010-08-03 15:12:04 PST -------
Speculative build fix: http://trac.webkit.org/changeset/64592
------- Comment #7 From 2010-08-03 15:43:47 PST -------
Speculative build fix part 2.
------- Comment #8 From 2010-08-03 15:44:23 PST -------
(In reply to comment #7)
> Speculative build fix part 2.

Oops! Forgot to add the link: http://trac.webkit.org/changeset/64596
------- Comment #9 From 2010-08-03 16:39:14 PST -------
Another build fix: http://trac.webkit.org/changeset/64600
------- Comment #10 From 2010-08-03 16:45:10 PST -------
This too: http://trac.webkit.org/changeset/64607