Closed Bug 1944835 Opened 1 year ago Closed 2 months ago

Unused object in PdfjsParent.sys.mjs

Categories

(Firefox :: PDF Viewer, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
149 Branch
Tracking Status
firefox149 --- fixed

People

(Reporter: arai, Assigned: henmofb)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

PdfjsParent.sys.mjs contains the following code, but the Svc variable is no longer used and it can be removed.

https://searchfox.org/mozilla-central/rev/dd8b64a6198ff599a5eb2ca096845ebd6997457f/toolkit/components/pdfjs/content/PdfjsParent.sys.mjs#39-45

var Svc = {};
XPCOMUtils.defineLazyServiceGetter(
  Svc,
  "mime",
  "@mozilla.org/mime;1",
  "nsIMIMEService"
);

Hey Tooru!

I'd be happy to make this simple change.

I am still very new to the Firefox OSS community, so apologies if I make some newbie mistakes here and there when submitting the patch.

Assignee: nobody → henmofb
Status: NEW → ASSIGNED
Attachment #9539927 - Attachment description: Bug 1944835 - Remove unused Svc object from PdfjsParent. r=arai → Bug 1944835 - Remove unused Svc object and unused getter definition from PdfjsParent. r=arai
Pushed by cdenizet@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/a82bfb869b9d https://hg.mozilla.org/integration/autoland/rev/590c80335aed Remove unused Svc object and unused getter definition from PdfjsParent. r=arai,calixte
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch
QA Whiteboard: [qa-triage-done-c150/b149]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: