The default bug view has changed. See this FAQ.

browser/components/nsBrowserContentHandler.js contains unecessary const

VERIFIED FIXED in Firefox 7

Status

()

Firefox
General
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: arno renevier, Assigned: arno renevier)

Tracking

Trunk
Firefox 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Hi,
browser/components/nsBrowserContentHandler.js constants definitions which are not used in the file.

const nsICategoryManager     = Components.interfaces.nsICategoryManager;

and 

const CONTRACTID_PREFIX = "@mozilla.org/uriloader/content-handler;1?type=";
(Assignee)

Comment 1

6 years ago
Created attachment 540123 [details] [diff] [review]
patch v1
Assignee: nobody → arno
Attachment #540123 - Flags: review?(gavin.sharp)
Attachment #540123 - Flags: review?(gavin.sharp) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
OS: Linux → All
Hardware: x86_64 → All
Landed on inbound.
Keywords: checkin-needed
Pushed:
http://hg.mozilla.org/mozilla-central/rev/05351157216d
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 7
Version: unspecified → Trunk

Comment 4

6 years ago
Mozilla/5.0 (X11; Linux i686; rv:7.0a1) Gecko/20110627 Firefox/7.0a1

Verified that the two lines were removed. Setting status to Verified Fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.