Closed
Bug 649211
Opened 15 years ago
Closed 15 years ago
PlacesDBUtils console output is broken
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla5
People
(Reporter: mak, Assigned: mak)
References
Details
(Keywords: regression, Whiteboard: [places-next-wanted])
Attachments
(1 file, 1 obsolete file)
|
3.11 KB,
patch
|
sdwilsh
:
review+
dveditz
:
approval2.0-
|
Details | Diff | Splinter Review |
This is a regression from bug 609286, most likely I want to backport this to FF4, and if we miss FF5, to it as well.
Maintenance works, but it does not print any result to the console, and this is a users support regression, since we can't have users report us results of running it.
Attachment #525260 -
Flags: review?(sdwilsh)
|
Assignee | |
Comment 1•15 years ago
|
||
I forgot to hg add the test, it's really simple but all other stuff is checked by the existing test, so this just checks that we output to the console.
Attachment #525260 -
Attachment is obsolete: true
Attachment #525260 -
Flags: review?(sdwilsh)
Attachment #525262 -
Flags: review?(sdwilsh)
|
|
Assignee | |
Updated•15 years ago
|
Flags: in-testsuite?
Whiteboard: [places-next-wanted]
|
|
Assignee | |
Updated•15 years ago
|
Blocks: 609286
Keywords: regression
|
||
Comment 2•15 years ago
|
||
Comment on attachment 525262 [details] [diff] [review]
patch v1.1
r=sdwilsh
Attachment #525262 -
Flags: review?(sdwilsh) → review+
|
|
Assignee | |
Comment 3•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
|
|
Assignee | |
Comment 4•15 years ago
|
||
Comment on attachment 525262 [details] [diff] [review]
patch v1.1
The patch is not scary and has a test. On Firefox 4 PlacesDBUtils works but it doesn't return any log to the user, we use these logs to figure out issues at the database level, so, as it is now, users can't report back problems to us.
I'd like to fix this in Firefox 4 that is the only version with a broken maintenance.
Attachment #525262 -
Flags: approval2.0?
|
||
Comment 5•14 years ago
|
||
Comment on attachment 525262 [details] [diff] [review]
patch v1.1
not planning a further 4.0.x releases
Attachment #525262 -
Flags: approval2.0? → approval2.0-
You need to log in
before you can comment on or make changes to this bug.
Description
•