Bug 122567 – Add support for minsize/maxsize attributes

• Component: MathML
• Status: RESOLVED FIXED

[Frédéric Wang (:fredw)]

The minsize/maxsize attributes allow to specify minimum and maximum size for vertical operators. This should be easy now that MathML length parsing is implemented.

See test 46 and 47 of the MathML Acid 3 test:
http://fred-wang.github.io/AcidTestsMathML/acid3/description.html

[Frédéric Wang (:fredw)]

Created attachment 225149 [details]
Patch

[Build Bot]

Comment on attachment 225149 [details]
Patch

Attachment 225149 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5399205568839680

New failing tests:
svg/masking/mask-negative-scale.svg
mathml/presentation/mo-stretch.html

[Build Bot]

Created attachment 225155 [details]
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-14  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[chris fleizach]

(In reply to comment #3)
> Created an attachment (id=225155) [details]
> Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2
> 
> The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
> Bot: webkit-ews-14  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

Looks like stretch operators are being stretched more than usual in this test. Does it just need to be updated?

[Build Bot]

Comment on attachment 225149 [details]
Patch

Attachment 225149 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5086643618840576

New failing tests:
svg/masking/mask-negative-scale.svg
mathml/presentation/mo-stretch.html

[Build Bot]

Created attachment 225159 [details]
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Frédéric Wang (:fredw)]

(In reply to comment #4)

> Looks like stretch operators are being stretched more than usual in this test. Does it just need to be updated?

I guess so but I can not see the screenshot. I have not been updating anything on this test but the Mac text reference, since for some reason that was the only thing the bots complained about.

The patch switches from int to LayoutUnit for stretch sizes so I expect that might changes slightly the size. Also, the default minsize (1em) is now taken into account so the small operators may be a bit larger.

[chris fleizach]

Created attachment 225160 [details]
image diff

diff looks significant from this image

[Frédéric Wang (:fredw)]

(In reply to comment #8)
> Created an attachment (id=225160) [details]
> image diff
> 
> diff looks significant from this image

I think this is a consequence of

https://bugs.webkit.org/show_bug.cgi?id=119043 (that fixes stretchy error on the top)

and 

https://bugs.webkit.org/show_bug.cgi?id=124827 (that makes these fences symmetric per the op dictionary)

but the reference has not been updated since the bots only attached the Mac text output.

[Build Bot]

Comment on attachment 225149 [details]
Patch

Attachment 225149 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6439513119260672

New failing tests:
svg/masking/mask-negative-scale.svg
mathml/presentation/mo-stretch.html

[Build Bot]

Created attachment 225162 [details]
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-04  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Frédéric Wang (:fredw)]

Created attachment 225164 [details]
Patch

[Frédéric Wang (:fredw)]

Committed r164700: <http://trac.webkit.org/changeset/164700>

[Frédéric Wang (:fredw)]

Mass change: add WebExposed keyword to help MDN documentation.