Closed Bug 1000592 Opened 7 years ago Closed 5 years ago
Enable CSS Grid by default in non-release channels
+++ This bug was initially created as a clone of Bug #1000431 +++ At the same time we should also remove: https://hg.mozilla.org/mozilla-central/diff/3038a5a34a75/testing/profiles/prefs_general.js
Summary: Enable "layout.css.grid.enabled" by default in non-release channels → Enable CSS Grid by default in non-release channels
All the listed blocking bugs now have patches (some still pending review). So I think we're ready to enable Grid in Nightly and Developer Edition builds, as soon as those lands.
(In reply to Mats Palmgren (:mats) from comment #0) > At the same time we should also remove: > https://hg.mozilla.org/mozilla-central/diff/3038a5a34a75/testing/profiles/prefs_general.js Note: the patch is missing this suggested testing/profiles/prefs_general.js tweak. But I think that's correct, because we'll still need this line for release builds (or else tests would start failing there when this bug's patch got merged to a RELEASE_BUILD).
Attachment #8675730 - Flags: review?(dholbert) → review+
I think we can (and should) remove that bit from the testing profile actually. Why would tests start failing without it?
Sorry, I should've said "might start failing" (if we have tests that assume grid is enabled -- which we could have, but maybe/probably we don't) r=me with the testing profile tweak added, as long as you've verified that tests pass with that (and without the all.js pref-flip). (Might really be better to land the testing profile tweak first, as its own patch, so that we get a full TreeHerder cycle with the test-profile pref-enabled configuration that we'll end up having on Release. But I suppose a Try run of just that tweak would do too.)
Actually, it's probably better to leave that as is so that we have test coverage for Grid on those channels too, even though it's not enabled by default (the Grid test manifest enables it unconditionally for those tests at least). We can remove it at the same time we let it ride the trains. I filed bug 1217086 and made a note there so we don't forget.
(In reply to Carsten Book [:Tomcat] from comment #9) > https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/282e4426f1e1 backed this out on request from mats from b2g v2.5 in http://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/0869ace8c965
You need to log in before you can comment on or make changes to this bug.