Closed
Bug 403817
Opened 17 years ago
Closed 17 years ago
nsDownloadManager.cpp #includes nsTArray.h but doesn't use it
Categories
(Toolkit :: Downloads API, defect)
Toolkit
Downloads API
Tracking
()
VERIFIED
FIXED
mozilla1.9beta3
People
(Reporter: sdwilsh, Assigned: rahul)
Details
(Whiteboard: [good first bug])
Attachments
(1 file)
960 bytes,
patch
|
Gavin
:
review+
mconnor
:
approval1.9+
|
Details | Diff | Splinter Review |
Reporter | ||
Updated•17 years ago
|
Whiteboard: [good first bug]
Assignee | ||
Comment 1•17 years ago
|
||
Attachment #292024 -
Flags: review?(gavin.sharp)
Updated•17 years ago
|
Attachment #292024 -
Flags: review?(gavin.sharp) → review+
Updated•17 years ago
|
Assignee: nobody → rahul
Keywords: checkin-needed
Updated•17 years ago
|
Keywords: checkin-needed
Updated•17 years ago
|
Attachment #292024 -
Flags: approval1.9?
Comment 2•17 years ago
|
||
Comment on attachment 292024 [details] [diff] [review] Unused #include removed from Download Manager a=mconnor on behalf of drivers, thanks for the patch!
Attachment #292024 -
Flags: approval1.9? → approval1.9+
Updated•17 years ago
|
Keywords: checkin-needed
Comment 3•17 years ago
|
||
Checking in toolkit/components/downloads/src/nsDownloadManager.cpp; /cvsroot/mozilla/toolkit/components/downloads/src/nsDownloadManager.cpp,v <-- nsDownloadManager.cpp new revision: 1.151; previous revision: 1.150 done
Status: NEW → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 M11
Comment 4•17 years ago
|
||
Verified FIXED using LXR. http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=nsDownloadManager.cpp&branch=1.152&root=/cvsroot&subdir=mozilla/toolkit/components/downloads/src&command=DIFF_FRAMESET&rev1=1.150&rev2=1.151
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•