Closed
Bug 380958
Opened 17 years ago
Closed 17 years ago
Strict Warning: PC_setBookmarksToolbarFolder does not always return a value
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: sdwilsh, Assigned: stevewon)
Details
Attachments
(1 file, 1 obsolete file)
1001 bytes,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
I didn't find an existing bug, so here's hoping it isn't a dupe. This is from the nightly: Warning: function PC_setBookmarksToolbarFolder does not always return a value Source File: chrome://browser/content/places/controller.js Line: 893, Column: 2 Source Code: },
Updated•17 years ago
|
Assignee: nobody → swon
Assignee | ||
Comment 1•17 years ago
|
||
Attachment #265171 -
Flags: review?(dietrich)
Comment 2•17 years ago
|
||
thanks steve. a couple of changes: 1. please use spaces instead of tabs (see http://developer.mozilla.org/en/docs/JavaScript_style_guide for that and other guidelines) 2. please indent lined up with the line above your change
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•17 years ago
|
||
Attachment #265171 -
Attachment is obsolete: true
Attachment #265181 -
Flags: review?(dietrich)
Attachment #265171 -
Flags: review?(dietrich)
Comment 4•17 years ago
|
||
Comment on attachment 265181 [details] [diff] [review] Patch modified looks good, thanks :)
Attachment #265181 -
Flags: review?(dietrich) → review+
Updated•17 years ago
|
Whiteboard: [checkin needed]
Reporter | ||
Comment 5•17 years ago
|
||
This was fixed by Bug 381335 before it got checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Comment 6•17 years ago
|
||
verified with Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a7pre) Gecko/2007070504 Minefield/3.0a7pre
Status: RESOLVED → VERIFIED
Comment 7•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•