Bug 155168 - Refactor RenderMathMLSpace to avoid using flexbox
Summary: Refactor RenderMathMLSpace to avoid using flexbox
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: MathML (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Frédéric Wang (:fredw)
URL:
Keywords:
Depends on:
Blocks: 153991
  Show dependency treegraph
 
Reported: 2016-03-08 06:19 PST by Frédéric Wang (:fredw)
Modified: 2016-04-20 06:25 PDT (History)
6 users (show)

See Also:


Attachments
Patch (5.48 KB, patch)
2016-03-08 06:24 PST, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch (6.70 KB, patch)
2016-03-11 01:27 PST, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch (6.71 KB, patch)
2016-03-31 08:24 PDT, Frédéric Wang (:fredw)
mrobinson: review+
Details | Formatted Diff | Diff
Patch (5.97 KB, patch)
2016-04-20 05:47 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2016-03-08 06:19:39 PST
To prepare for bug 153991, let's do as in other classes and implement RenderMathMLSpace::computePreferredLogicalWidths, RenderMathMLSpace::layoutBlock and RenderMathMLSpace::firstLineBaseline
Comment 1 Frédéric Wang (:fredw) 2016-03-08 06:24:43 PST
Created attachment 273293 [details]
Patch
Comment 2 Frédéric Wang (:fredw) 2016-03-11 01:27:28 PST
Created attachment 273707 [details]
Patch

I'm uploading updated version of the MathML refactoring patches to solve conflicts after bug 154388 and bug 155005 ; and to do other minor changes.
Comment 3 Frédéric Wang (:fredw) 2016-03-31 08:24:01 PDT
Created attachment 275291 [details]
Patch
Comment 4 Martin Robinson 2016-04-04 11:57:51 PDT
Comment on attachment 275291 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=275291&action=review

> LayoutTests/ChangeLog:8
> +        * mathml/presentation/mspace-prefered-width-expected.html: Update the expectation so that the logical width and preferred width are both equal to the one specified by the width attribute.

Please break lines at 100 characters.

> Source/WebCore/ChangeLog:8
> +        No new tests, already covered by existing tests. The behavior of mspace-prefered-width-expected is not specified by the MathML recommendation, we update that test to match our new behavior.

Ditto.
Comment 5 Frédéric Wang (:fredw) 2016-04-20 05:47:09 PDT
Created attachment 276819 [details]
Patch

Rebasing patch.
Comment 6 Frédéric Wang (:fredw) 2016-04-20 06:09:00 PDT
Removing dependencies as we don't really have to wait for bug 155018/155019 to take this patch.
Comment 7 Frédéric Wang (:fredw) 2016-04-20 06:25:31 PDT
Committed r199771: <http://trac.webkit.org/changeset/199771>