Closed Bug 11152 Opened 21 years ago Closed 20 years ago

nsBrowserAppCore.cpp code cleanup

Categories

(SeaMonkey :: UI Design, defect, P2, minor)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: waterson, Assigned: law)

References

()

Details

(Whiteboard: [HELP WANTED])

Stumbled across this gem at the above URL. Probably don't need to convert to a
C string only to convert back to unicode two lines later. Looks this is a
product of too many chefs in the kitchen :-)
Component: Browser-General → XPApps
Priority: P3 → P2
Target Milestone: M10
Severity: normal → minor
Status: NEW → ASSIGNED
The code in question has changed so the URL doesn't seem to point to the bogus
code any longer.  I'm not quite sure what to look for in the nearby code.  Do
you happen to remember more about it (what function, what the current line
number is, what the code looked like)?  I'll go back and fetch this file by date
and see what was at that line number, but it's not a top priority and it might
save time if you could provide a more up-to-date pointer.  Thanks.
Target Milestone: M11 → M15
Moving this off to a milestone beyond the foreseeable future...
Whiteboard: [HELP WANTED]
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
nsBrowserAppCore is history so I'm marking this fixed.  The code in question
might well be in the replacement nsBrowserInstance.cpp but I have no way of
knowing since I still don't know what the code was.
QA Contact: leger → don
don...ok to mark Verified?
QA Contact: don → sairuh
rubberstamp.
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.