In r201042 the runtime flags were enabled by default as Safari Technology Preview was lacking a UI to modify them. Now that it has a UI to enable/disable the flags we don't need that Grid Layout is enabled by default anymore.
Created attachment 291588 [details] Patch
Comment on attachment 291588 [details] Patch Clearing flags on attachment: 291588 Committed r207402: <http://trac.webkit.org/changeset/207402>
All reviewed patches have been landed. Closing bug.
This change caused some buttons in Web Inspector to break their position!! That said, I don't think we should roll this out, I think this is just exposing an issue that this runtime flag is affecting some things that it should be affecting. I'll investigate.
(In reply to comment #4) > This change caused some buttons in Web Inspector to break their position!! > That said, I don't think we should roll this out, I think this is just > exposing an issue that this runtime flag is affecting some things that it > should be affecting. I'll investigate. Is web inspector using grid layout? Sounds weird specially because we have not shipped it yet.
(In reply to comment #5) > (In reply to comment #4) > > This change caused some buttons in Web Inspector to break their position!! > > That said, I don't think we should roll this out, I think this is just > > exposing an issue that this runtime flag is affecting some things that it > > should be affecting. I'll investigate. > > Is web inspector using grid layout? Sounds weird specially because we have > not shipped it yet. The bug #163572 has been filed precisely to track this issue. The problem is not grid itself, but the CSS Box Alignment properties it shares with Flexbox. Due to the different behavior of the new syntax, we need to use the runtime and compile flags to implement both code paths. BTW, there is already a proposed fix for bug #163572, so we can continue discussing the details there.