Closed Bug 1263631 Opened 4 years ago Closed 4 years ago

Update web-platorm-tests to 8d896c2015ab1e50ad00a0013700f87813d9364c

Categories

(Testing :: web-platform-tests, defect)

defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: jgraham, Unassigned)

References

(Blocks 2 open bugs)

Details

Blocks: 1263643
Blocks: 1261053
Hi jgraham, looks like testing/web-platform/meta/compat/ folder has been removed in this bug. Do you think this is normal while updating web-platorm-tests? If deleting ini files is the right behavior, should I still add ini files for newly added reftest next time?
Flags: needinfo?(james)
Oooh, that looks like a bug. It shouldn't delete non-empty ini files e.g. ones specifying prefs. Although I do wonder why doing so didn't affect the result of the test. Are those prefs now set by default?
Flags: needinfo?(james) → needinfo?(jeremychen)
(In reply to James Graham [:jgraham] from comment #4)
> Oooh, that looks like a bug. It shouldn't delete non-empty ini files e.g.
> ones specifying prefs. Although I do wonder why doing so didn't affect the
> result of the test. Are those prefs now set by default?

Yes, the pref is default true except for release build.
https://dxr.mozilla.org/mozilla-central/source/modules/libpref/init/all.js#2397
Flags: needinfo?(jeremychen)
Blocks: 1264269
You need to log in before you can comment on or make changes to this bug.