Re-enable flexbox pref (by default) in release builds

VERIFIED FIXED in mozilla22

Status

()

Core
Layout
VERIFIED FIXED
4 years ago
3 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

({dev-doc-complete})

Trunk
mozilla22
dev-doc-complete
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(relnote-firefox 22+)

Details

(Whiteboard: [DocArea=CSS])

Attachments

(1 attachment)

Filing this bug on reverting the changes in bug 841873, once flexbox is sufficiently stable to ship preffed-on by default in release builds.

(Any bugs that should block flexbox from being shipped in release builds should be marked as blocking this bug.)
Some other bugs that we may or may not want to block on:
 - bug 782441
 - bug 783470
 - bug 811024
 - bug 812687
For RelMan - I bumped the note that was in 20 back to 22 for now, marking relnote-firefox? so we keep this note on our radar and get it back into the right notes.
relnote-firefox: --- → ?
Depends on: 848539
No longer blocks: 841827, 812822, 821775, 841847
Depends on: 841827, 812822, 821775, 841847
Depends on: 782441
Blocks: 798592
Depends on: 851379

Comment 3

4 years ago
We'll relnote once resolved. Please renominate at that time.
relnote-firefox: ? → ---
Created attachment 730590 [details] [diff] [review]
fix v1
Assignee: nobody → dholbert
Attachment #730590 - Flags: review?(dbaron)
Comment on attachment 730590 [details] [diff] [review]
fix v1

r=dbaron
Attachment #730590 - Flags: review?(dbaron) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/845d28ab458e
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/845d28ab458e
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Keywords: dev-doc-needed

Comment 8

4 years ago
Is flexbox impl stable now? There are still a couple of open bugs left (excluding multi-line flexbox – I guess support for multi-line support can be checked via @supports?) … maybe wait one more release before pushing this through?
(In reply to Florian Bender from comment #8)
> Is flexbox impl stable now?

I believe so, though testing on nightly/aurora/beta will verify that.

> There are still a couple of open bugs left
> (excluding multi-line flexbox – I guess support for multi-line support can
> be checked via @supports?) … maybe wait one more release before pushing this
> through?

Do you have any specific bugs that you think should block this?

All the bugs that I marked as blocking this one have been resolved, except for bug 851379 which was a very late-breaking spec-change. (which we wouldn't block our release on, & which we can likely backport, per bug 851379 comment 5)

There are a several remaining fuzzer bugs that Jesse has filed, which I intend to look into, but their fixes are likely to be minimal and backportable.
(In reply to Daniel Holbert [:dholbert] from comment #9)
> (In reply to Florian Bender from comment #8)
> > Is flexbox impl stable now?
> 
> I believe so, though testing on nightly/aurora/beta will verify that.

(To clarify that: note that flexbox is already on by default in Nightly/Aurora builds, and has been on there for months. This bug's patch only affects behavior in beta & release builds.)
(and yeah, @supports is a good way to test for multi-line support -- or, in script, checking for the presence of someElement.style.flexWrap)

Comment 12

4 years ago

(In reply to Daniel Holbert [:dholbert] from comment #10)
> This bug's patch only affects behavior in beta & release builds.)
Just edited Fx 20 for Devs the 2nd time, now that I know that I didn't mess with the prefs and wasn't the only Beta user without flexbox ;).

(In reply to Daniel Holbert [:dholbert] from comment #9)
> Do you have any specific bugs that you think should block this?
Not really, but what about some of the depending and dependant bugs linked in bug 666041? E. g. Bug 799699, Bug 783470 (indicates this is not necessary before shipping, but it's not testable from web code), Bug 811024 (also not testable, but probably not easy to implement), and Bug 734525 (the discussion is hopefully resolved by now).

Updated

4 years ago
relnote-firefox: --- → ?

Comment 13

4 years ago
Verified as fixed on the 04/03 Aurora (20130403004013), on Windows 7 64bit, Ubuntu 12.10 32bit and Mac OSX 10.7.5 64bit.

The pref is true by default, there are no error messages about flex not being known or supported, and pages with flex elements load and render fine.
Status: RESOLVED → VERIFIED

Updated

4 years ago
relnote-firefox: ? → 22+
This bug has been listed as part of the Aurora 22 release notes in either [1], [2], or both. If you find any of the information or wording incorrect in the notes, please email release-mgmt@mozilla.com asap. Thanks!

[1] https://www.mozilla.org/en-US/firefox/22.0a2/auroranotes/
[2] https://www.mozilla.org/en-US/mobile/22.0a2/auroranotes/

Updated

4 years ago
Blocks: 858305
Depends on: 879540

Comment 15

4 years ago
I am new to bug reporting on Firefox so please forgive my ignorance - I have Firefox stable version 23 and the `layout.css.flexbox.enabled` option still defaults to false, though I expected it to be true since this bug is marked as fixed for v22.  What am I missing?
I just downloaded a fresh Firefox 23.01 build from [1] and I started it up with a fresh profile, and I verified that the pref defaults to "true" for me. So I think all is well.

JZ, could you create a new Firefox profile as described at [2], and see if you still see the problem in the fresh profile?

[1] https://www.mozilla.org/en-US/products/download.html
[2] https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles

Comment 17

4 years ago
As you suggested, when I started Firefox using a fresh profile the value was `true`. I don't mind manually adjusting the value in my default profile but I am wondering under what circumstances it would have been stuck on `false` - and for how many other users this is the case - is it possible that I manually toggled it back and forth earlier and for that reason the default value was not changed when I last updated Firefox?
(In reply to JZ from comment #17)
> As you suggested, when I started Firefox using a fresh profile the value was
> `true`.

Good, thanks for trying that -- that's great to hear.

> I don't mind manually adjusting the value in my default profile but
> I am wondering under what circumstances it would have been stuck on `false`
> [...] is it possible that I
> manually toggled it back and forth earlier and for that reason the default
> value was not changed when I last updated Firefox?

It doesn't seem like that would have done it, no. I tried various ways of toggling a boolean pref away from its default and then flipping it back again (either with a double-click, right-click "Toggle", or right-click & "Reset"), sometimes quitting & relaunching between the togglings.  In each case, after I'd restored the pref to the default value, the pref was no longer present in my profile's "prefs.js" file.  So that tells me that simply turning the pref on and off wouldn't have caused your problem.

The only way I know of to get the pref persistently set to "false" would be to manually set it to false, in a browser that has it defaulting to "true".  Maybe you did that at some point while running an Aurora or Nightly prerelease build? (perhaps you did so, thinking that you were switching it back to the default value, since that was the default in release builds, but it was *not* the default in Aurora/Nightly builds, which meant the "false" was persistently stored in your prefs.js file)

Anyway; given that this is the first I've heard of this, and given that this seems to be fine for profiles that haven't been tweaked, I think we're OK here, and we probably shouldn't clutter up this bug with more discussion. If you figure out more about what happened in your case, and it seems to be a bug, then it'd probably be worth filing a Prefs bug[1] and following up there. though.

[1] https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&component=Preferences
Blocks: 936100
Whiteboard: [DocArea=CSS]
This has been documented long ago.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.