1426747 - [css2] Remove the IE compat quirk for <table align=left>

Status: RESOLVED FIXED

(Core :: Layout: Floats, defect, P3)

Description

[Emilio Cobos Álvarez (:emilio)]

https://searchfox.org/mozilla-central/rev/22c55eb7b7e6494a8615a7af3b613ff899d2cdba/layout/generic/BlockReflowInput.cpp#843

Let's see if we an kill it.

Comment 1

[Jet Villegas (inactive)]

[ Triage 2017/02/20: P3 ]

Comment 2

[Mats Palmgren (inactive)]

Agreed. Neither Chrome, nor Safari, implements this quirk. (I didn't check EdgeHTML, but it's mostly obsolete already.)

I took a stab at it:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a7447b5900c0cf84714111d0297efe8b5a1ecc49

Comment 3

[Mats Palmgren (inactive)]

Interestingly, my patch made layout/reftests/bugs/50630-2.html fail, which isn't using align=left but <table style="float:left;"> so it appears our quirk here unintentionally also affected other content.

Again, we currently disagree with Chrome on the layout of this test and the patch makes us compatible.

Comment 4

[Mats Palmgren (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=31b716bdf024f2cef1733622f4f721cdcdb876e4

Comment 5

[Mats Palmgren (inactive)]

Attachment: Bug 1426747 - Remove the IE compat quirk for <table align=left>. r=emilio

Chrome and Safari don't implement this quirk.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c30d38db0a5
Remove the IE compat quirk for <table align=left>.  r=emilio

Comment 7

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26226 for changes under testing/web-platform/tests

Comment 8

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/8c30d38db0a5

Comment 9

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot