The default bug view has changed. See this FAQ.

Move Bookmarks dialog broken

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Peter B. Shalimoff, Assigned: Robert Kaiser)

Tracking

Trunk
seamonkey2.1b1

Firefox Tracking Flags

(blocking-seamonkey2.1 final+)

Details

Attachments

(3 attachments)

(Reporter)

Description

7 years ago
User-Agent:       Mozilla/5.0 (Windows NT 5.2; rv:2.0b4pre) Gecko/20100817 SeaMonkey/2.1a3
Build Identifier: Mozilla/5.0 (Windows NT 5.2; rv:2.0b4pre) Gecko/20100817 SeaMonkey/2.1a3

The "Choose Folder" seems to be broken:
1) It doesn't display the existing folders.
2) Clicking its "New Folder" button does nothing visible (and there are no errors in the Error Console) - see cfdlg.png screenshot.
3) Clicking its "OK" buttons results in an assertion failure (see cfdlg.ok_assert.png screenshot):
   ASSERT: selectedNode must be set in a single-selection tree with initial selection set
   Stack Trace: 
   0:MBD_onOK([object Event])
   1:anonymous([object Event])
   2:_fireButtonEvent(accept)
   3:_doButtonCommand(accept)
   4:_handleButtonCommand([object XULCommandEvent])


Reproducible: Always

Steps to Reproduce:
1. Open Bookmarks Manager.
2. Select (hilight) a bookmark.
3. Select "Edit -> Move..." from the menu.
4. Click the "New Folder" or the "OK" button.

Actual Results:  
See the Details.


Expected Results:  
The dialog should display the existing folders.
Clicking the "New Folder" should open a folder creation dialog.
Clicking the "OK" button should, um, just work like an ordinary OK button - close the dialog and apply any changes made. :)
(Reporter)

Comment 1

7 years ago
Created attachment 469195 [details]
Screenshot: cfdlg.png

Show an empty "Choose Folder" dialog and Error Console with no errors after clicking the "New Folder" button.
(Reporter)

Comment 2

7 years ago
Comment on attachment 469195 [details]
Screenshot: cfdlg.png

Show an empty "Choose Folder" dialog and Error Console with no errors after clicking the "New Folder" button.
(Reporter)

Comment 3

7 years ago
Created attachment 469198 [details]
Screenshot: cfdlg.ok_assert.png

Shows an assertion failure after clicking the "OK" button.
(Reporter)

Comment 4

7 years ago
Comment on attachment 469198 [details]
Screenshot: cfdlg.ok_assert.png

Shows an assertion failure after clicking the "OK" button.
(Reporter)

Comment 5

7 years ago
Sorry for the dup comments - I misunderstood the "Comment" text field in the attachment creation form.
(Assignee)

Updated

7 years ago
Summary: Bookmark folder selection dialog is broken. → Move Bookmarks dialog broken

Comment 6

7 years ago
Build identifier: Mozilla/5.0 (X11; Linux x86_64; rv:2.0b5pre) Gecko/20100826 SeaMonkey/2.1b1pre

I created a new profile and the error still appears. I didn't import new bookmarks.

When I click OK, Linux command line display:

[root@venuse seamonkey]#   /usr/local/seamonkey/2_1_x_Eng/seamonkey -no-remote -P 2.1
ASSERT: selectedNode must be set in a single-selection tree with initial selection set

After the closing of all windows Bookmark Manager Error Console display:

Error: An error occurred updating the cmd_selectAll command: [Exception... "'[JavaScript Error: "this._view.result is null" {file: "chrome://communicator/content/places/controller.js" line: 181}]' when calling method: [nsIController::isCommandEnabled]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goUpdateCommand :: line 80"  data: yes]
Source File: chrome://global/content/globalOverlay.js
Line: 86

Updated

7 years ago
blocking-seamonkey2.1: --- → ?
Could reproduce it with Mozilla/5.0 (X11; Linux x86_64; rv:2.0b5pre) Gecko/20100827 SeaMonkey/2.1b1pre

I get also a warning opening the move bookmark dialog, but don't know if it's related to this bug:

WARNING: Context has no global.: file <path>/comm-central/mozilla/dom/base/nsJSEnvironment.cpp, line 2518
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows Server 2003 → All
Hardware: x86 → All
Version: unspecified → Trunk
Blocks: 498596
(In reply to comment #7)
> WARNING: Context has no global.: file
> <path>/comm-central/mozilla/dom/base/nsJSEnvironment.cpp, line 2518

That is usually the sign of a much larger problem...

Have either of you tested on an optimized build, and can you test with the newest nightly?

Do either of you have access to a non-linux machine and can verify if it happens there too? [If not I'll check on windows later]

Comment 9

7 years ago
Mozilla/5.0 (Windows NT 5.1; rv:2.0b6pre) Gecko/20100907 SeaMonkey/2.1b1pre

When I open Bookmark Manager, Error Console display:

CoCreateInstance() FAILED.

(In reply to comment #0)
> The "Choose Folder" seems to be broken:
> 1) It doesn't display the existing folders.
> 2) Clicking its "New Folder" button does nothing visible (and there are no
> errors in the Error Console) - see cfdlg.png screenshot.
> 3) Clicking its "OK" buttons results in an assertion failure (see
> cfdlg.ok_assert.png screenshot):
>    ASSERT: selectedNode must be set in a single-selection tree with initial
> selection set
>    Stack Trace: 
>    0:MBD_onOK([object Event])
>    1:anonymous([object Event])
>    2:_fireButtonEvent(accept)
>    3:_doButtonCommand(accept)
>    4:_handleButtonCommand([object XULCommandEvent])

It is diplay too.
(In reply to comment #9)

> When I open Bookmark Manager, Error Console display:
> 
> CoCreateInstance() FAILED.

I also get this on every occasion I use a menu, so probably not related to the problem here.

Comment 11

7 years ago
Aqualon, have I make stacktrace with WinDbg?

Comment 12

7 years ago
(In reply to comment #6)
> After the closing of all windows Bookmark Manager Error Console display:
> 
> Error: An error occurred updating the cmd_selectAll command: [Exception...
> "'[JavaScript Error: "this._view.result is null" {file:
> "chrome://communicator/content/places/controller.js" line: 181}]' when calling
> method: [nsIController::isCommandEnabled]"  nsresult: "0x80570021
> (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame ::
> chrome://global/content/globalOverlay.js :: goUpdateCommand :: line 80"  data:
> yes]
> Source File: chrome://global/content/globalOverlay.js
> Line: 86

Mozilla/5.0 (Windows NT 5.1; rv:2.0b6pre) Gecko/20100912 Firefox/4.0b6pre SeaMonkey/2.1b1pre

It display only when didn't open Error console window during the test Bookmark Manager.
(In reply to comment #6)
> Error: An error occurred updating the cmd_selectAll command: [Exception...
> "'[JavaScript Error: "this._view.result is null" {file:
> "chrome://communicator/content/places/controller.js" line: 181}]' when calling
> method: [nsIController::isCommandEnabled]"  nsresult: "0x80570021
> (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame ::
> chrome://global/content/globalOverlay.js :: goUpdateCommand :: line 80"  data:
> yes]
> Source File: chrome://global/content/globalOverlay.js
> Line: 86

FWIW, i get the same in Mozilla/5.0 (Windows NT 5.1; rv:2.0b7pre) Gecko/20100919 Firefox/4.0b7pre, of course for a different File:
"chrome://browser/content/places/controller.js" line: 179
Bug in the shared Places Code?
(Assignee)

Comment 14

7 years ago
(In reply to comment #13)
> Bug in the shared Places Code?

At least that's the same code. Please file a bug on Firefox bookmarks and history for that, though, and note it there, as we'll need patches on both sides, reviewed by both teams, in any case.

Does the Move Bookmarks window work in Firefox trunk?
(Assignee)

Comment 15

7 years ago
Created attachment 478538 [details] [diff] [review]
fix it!

It might be better to link the correct CSS file to get the XBL applied...

Addressing review to Ian as he reviewed the original patch. ;-)
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #478538 - Flags: review?(iann_bugzilla)
(Assignee)

Updated

7 years ago
blocking-seamonkey2.1: ? → final+
Target Milestone: --- → seamonkey2.1b1

Updated

7 years ago
Attachment #478538 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 16

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/4f719059999e
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 17

7 years ago
Mozilla/5.0 (Windows NT 5.1; rv:2.0b7pre) Gecko/20100926 Firefox/4.0b7pre SeaMonkey/2.1b1pre

It works fine now
(In reply to comment #9)
> When I open Bookmark Manager, Error Console display:
> 
> CoCreateInstance() FAILED.
It's trying to detect nVidia 3D Vision. See bug 598328.
You need to log in before you can comment on or make changes to this bug.