Closed Bug 614103 Opened 14 years ago Closed 14 years ago

Nuke nsPIPlacesHistoryListenersNotifier.idl

Categories

(Toolkit :: Places, defect)

Product:
Toolkit
Component:
Places
Platform:
x86
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: sdwilsh, Unassigned)

References

Details

(Whiteboard: [killthem][good first bug])

Attachments

(1 file)

patch
8.72 KB, patch
mak
: review-
Details | Diff | Splinter Review 
We should have gotten rid of it and our implementation in bug 552023, but it was missed.
Whiteboard: [killthem] → [killthem][good first bug]
Attached patch patch — — Splinter Review 
Apologies if I did anything wrong or stupid, it's my very first bug :)
Attachment #493541 - Flags: review?(sdwilsh)
hm, expiration is using this interface, I suspect there is something wrong in the request in comment 0.
Comment on attachment 493541 [details] [diff] [review]
patch

sorry, r- for now, till Shawn clarifies the request.
Attachment #493541 - Flags: review?(sdwilsh) → review-
That was rather silly of me.  I totally forgot to check expiration, and yeah, it uses this :(

Sorry about the wasted effort Matheus!  If you are interested in trying a different bug, I'd be happy to point you at some.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
No problem Shawn, and if you know any easy bugs I can work on please let me know!
Sorry for the delay.  You could take a look at bug 465299, bug 472530, or bug 554115.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: