Use common spelling for source map prefs

VERIFIED FIXED in Firefox 54

Status

()

Firefox
Developer Tools
P3
normal
VERIFIED FIXED
6 months ago
4 months ago

People

(Reporter: jryans, Assigned: jryans)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 54
Points:
---

Firefox Tracking Flags

(firefox54 verified)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

I want all source map prefs to appear when filtering about:config, so they need to use the same spelling.
"source-map" is used by 3 prefs, "sourcemap" is used by 1 pref.

Let's go with "source-map" for all of them.
Comment hidden (mozreview-request)

Comment 3

6 months ago
mozreview-review
Comment on attachment 8837860 [details]
Bug 1339992 - Use common spelling for all source map prefs.

https://reviewboard.mozilla.org/r/112854/#review114468

Thanks! 

This means all users that had set this preference will have to reset it, but it wasn't advertised anywhere so that's fine.
Attachment #8837860 - Flags: review?(jdescottes) → review+

Comment 4

6 months ago
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e76a72dceeea
Use common spelling for all source map prefs. r=jdescottes

Comment 5

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e76a72dceeea
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
status-firefox54: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54

Comment 6

4 months ago
I have reproduced this bug with Nightly 54.0a1 (2017-02-15) on Windows 10, 64 bit!

The bug's fix is now verified on Latest Beta 54.0b3

Build ID 	20170427091925
User Agent 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:54.0) Gecko/20100101 Firefox/54.0

[testday-20170428]
Status: RESOLVED → VERIFIED
status-firefox54: fixed → verified
You need to log in before you can comment on or make changes to this bug.