Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Drop the code that uses extensions.screenshots.system-disabled part 2 #3549

Closed
ghost opened this issue Sep 25, 2017 · 0 comments
Closed

Drop the code that uses extensions.screenshots.system-disabled part 2 #3549

ghost opened this issue Sep 25, 2017 · 0 comments
Assignees
@jaredhirsch
Labels
firefox Something that has to ship in the add-on in Firefox
Milestone
General Release 57

Comments

@ghost
Copy link

ghost commented Sep 25, 2017

This was originally #3283 but that was closed with just removing the default pref. We should remove the code in the add-on that looks at system-disabled altogether.
@ianb ianb added the firefox Something that has to ship in the add-on in Firefox label Sep 26, 2017
@ianb ianb added this to the General Release 57 milestone Sep 26, 2017
@jaredhirsch jaredhirsch self-assigned this Sep 27, 2017
jaredhirsch added a commit that referenced this issue Sep 27, 2017
@jaredhirsch
Fix #3549, remove code that checks for system-disabled pref
5726bfc 
Leave the branch/leaf checking code in place, both to minimize the size
of the patch, and to make it easy to observe a self-hosting pref I'd
like to add soon.
jaredhirsch added a commit that referenced this issue Sep 29, 2017
@jaredhirsch
Fix #3549, remove code that checks for system-disabled pref
95de2bb 
Leave the branch/leaf checking code in place, both to minimize the size
of the patch, and to make it easy to observe a self-hosting pref I'd
like to add soon.
jaredhirsch added a commit that referenced this issue Sep 29, 2017
@jaredhirsch
Fix #3549, remove code that checks for system-disabled pref
0737bd5 
Leave the branch/leaf checking code in place, both to minimize the size
of the patch, and to make it easy to observe a self-hosting pref I'd
like to add soon.
jaredhirsch added a commit that referenced this issue Sep 29, 2017
@jaredhirsch
Fix #3549, remove code that checks for system-disabled pref
82caa61 
Leave the branch/leaf checking code in place, both to minimize the size
of the patch, and to make it easy to observe a self-hosting pref I'd
like to add soon.
jaredhirsch added a commit that referenced this issue Oct 10, 2017
@jaredhirsch
Fix #3549, remove code that checks for system-disabled pref (#3584)
92376fd 
Leave the branch/leaf checking code in place, both to minimize the size
of the patch, and to make it easy to observe a self-hosting pref I'd
like to add soon.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jaredhirsch jaredhirsch

Labels
firefox Something that has to ship in the add-on in Firefox
Projects
None yet
Milestone
General Release 57
Development

No branches or pull requests
2 participants
@ianb @jaredhirsch