Closed
Bug 431535
Opened 16 years ago
Closed 16 years ago
Static item's context menu in bookmark menu is messed up
Categories
(Firefox :: Bookmarks & History, defect)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: zurtex, Assigned: mak)
References
Details
(Keywords: regression)
Attachments
(2 files)
2.68 KB,
image/png
|
Details | |
2.85 KB,
patch
|
asaf
:
review+
mconnor
:
approval1.9+
|
Details | Diff | Splinter Review |
1. Click on Bookmarks in the menu bar (between History and Tools) 2. Move mouse over the static "Bookmarks Toolbar" item 3. Right click on the static "Bookmarks Toolbar" item 4. Right click menu is a mess (see attached) Made invalid by bug 429469 ?
Flags: blocking-firefox3?
Reporter | ||
Comment 1•16 years ago
|
||
Clearing blocking for now, I suddenly found I could no longer duplicate it, testing on: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9pre) Gecko/2008042906 Minefield/3.0pre ID:2008042906 Will renom when I find a more reproducible STR.
Flags: blocking-firefox3?
Reporter | ||
Comment 2•16 years ago
|
||
Aha, found the problem, here's a STR that has 100% duplication: 1. Close down all instances of Firefox 2. Open up Firefox 3. Right click on any static item in the bookmarks menu (accessed from the menubar) Behavior changes when right clicking on a non-static item in the bookmarks menu Renominating now I can dupe 100% and have found the cause of why I couldn't before.
Flags: blocking-firefox3?
Summary: "Bookmark toolbar" menu's context menu messed up → Static item's context menu in bookmark toolbar messed up
Assignee | ||
Comment 3•16 years ago
|
||
Error: aNode is null Source File: file:///C:/Users/MaK77/Desktop/minefield/modules/utils.js Line: 388 Error: uncaught exception: [Exception... "'[JavaScript Error: "aNode is null" {file: "file:///C:/Users/MaK77/Desktop/minefield/modules/utils.js" line: 388}]' when calling method: [nsIController::isCommandEnabled]" nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)" location: "JS frame :: chrome://browser/content/places/controller.js :: updatePlacesCommand :: line 1490" data: yes] Source File: chrome://browser/content/places/menu.xml Line: 898 can you look out for a regression window?
Keywords: qawanted
Reporter | ||
Comment 4•16 years ago
|
||
Bug does NOT occur on: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9pre) Gecko/2008042705 Minefield/3.0pre ID:2008042705 Bug does occur on: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9pre) Gecko/2008042806 Minefield/3.0pre ID:2008042806 CVS check-ins in that time: http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&branch=HEAD&date=explicit&mindate=2008-04-27+05%3A00&maxdate=2008-04-28+06%3A00 (though I've never done a CVS search before, so double check I've got that right)
Assignee | ||
Comment 5•16 years ago
|
||
thank you, regressed in bug 428403, patch coming
Reporter | ||
Updated•16 years ago
|
Summary: Static item's context menu in bookmark toolbar messed up → Static item's context menu in bookmark menu is messed up
Assignee | ||
Comment 6•16 years ago
|
||
directly check if we have a valid places node selected, if not we use the current selectedNode (since in that case selectedNode is our parent). this works also for special empty (and for static items clearly)
Attachment #318610 -
Flags: review?(mano)
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [has patch][needs review mano]
Updated•16 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Updated•16 years ago
|
Comment 8•16 years ago
|
||
Comment on attachment 318610 [details] [diff] [review] patch r=mano
Attachment #318610 -
Flags: review?(mano) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #318610 -
Flags: approval1.9?
Assignee | ||
Updated•16 years ago
|
Whiteboard: [has patch][needs review mano] → [has patch][has reviews]
Comment 9•16 years ago
|
||
Comment on attachment 318610 [details] [diff] [review] patch a=mconnor on behalf of 1.9 drivers
Attachment #318610 -
Flags: approval1.9? → approval1.9+
Updated•16 years ago
|
Keywords: checkin-needed
Whiteboard: [has patch][has reviews] → [has patch][has reviews][has approval]
Comment 10•16 years ago
|
||
mozilla/browser/components/places/content/menu.xml 1.133 mozilla/browser/components/places/content/toolbar.xml 1.155
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [has patch][has reviews][has approval]
Target Milestone: --- → Firefox 3
Comment 11•16 years ago
|
||
Marco, the same happens when you click on an empty area within the Bookmarks Toolbar. Will this be also fixed with this patch?
Assignee | ||
Comment 12•16 years ago
|
||
yes, IIRC i've tested that too. it should work since in that zone you don't have a valid places node
Comment 13•16 years ago
|
||
This fixes the context menu for the Bookmarks Menu and even the mentioned issues for the Bookmarks Toolbar. Verified with: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008050606 Minefield/3.0pre ID:2008050606 Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9pre) Gecko/2008050619 Minefield/3.0pre and on Ubuntu 8.04 with the same build.
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Flags: in-litmus?
Comment 14•15 years ago
|
||
Test cases were updated on litmus FFt's for 3.x test runs. For 3.0, https://litmus.mozilla.org/show_test.cgi?id=7599 For 3.1, https://litmus.mozilla.org/show_test.cgi?id=7480
Flags: in-litmus? → in-litmus+
Comment 15•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
•