Closed Bug 1635068 Opened 4 years ago Closed 3 years ago

remove dead pref extensions.screenshots.upload-disabled and unused function isUploadDisabled()

Categories

(Firefox :: Screenshots, enhancement)

77 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: earthlng, Assigned: falgunimst95)

Details

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Firefox/68.0

Steps to reproduce:

The Screenshot extension's upload functionality was removed a while ago and extensions.screenshots.upload-disabled is now a dead pref. [1]

The custom experimental API function isUploadDisabled() isn't used anymore either and can also get removed. [2]

Good first bug

[1] https://searchfox.org/mozilla-central/search?q=upload-disabled
[2] https://searchfox.org/mozilla-central/search?q=isUploadDisabled

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:76.0) Gecko/20100101 Firefox/76.0

Hi,

I will move this enhancement over to a component so developers can take a look over it. If this is not the correct component please feel free to change it to an appropriate one.

Thanks for your suggestion.

Status: UNCONFIRMED → NEW
Component: Untriaged → Preferences
Ever confirmed: true
Product: Firefox → Toolkit
Component: Preferences → Screenshots
Product: Toolkit → Firefox
Attached patch 1635068.patchSplinter Review

This is my 1st patch. Not sure what to select under flags etc.

If I did something wrong please let me know. Also don't know how to proceed now... any help is appreciated.

Assignee: nobody → falgunimst95
Status: NEW → ASSIGNED
Attachment #9215724 - Attachment description: Bug 1635068: fix remove dead pref extensions.screenshots.upload-disabled and unused function isUploadDisabled() → Bug 1635068: fix removed dead pref extensions.screenshots.upload-disabled and unused function isUploadDisabled()
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d5121e6b3b60
fix removed dead pref extensions.screenshots.upload-disabled and unused function isUploadDisabled() r=emalysz
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: