Peter B. Shalimoff Reporter
	Description • 14 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. :)

	Peter B. Shalimoff Reporter
	Comment 1 • 14 years ago

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

	Peter B. Shalimoff Reporter
	Comment 2 • 14 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.

	Peter B. Shalimoff Reporter
	Comment 3 • 14 years ago

Shows an assertion failure after clicking the "OK" button.

	Peter B. Shalimoff Reporter
	Comment 4 • 14 years ago

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

Shows an assertion failure after clicking the "OK" button.

	Peter B. Shalimoff Reporter
	Comment 5 • 14 years ago

Sorry for the dup comments - I misunderstood the "Comment" text field in the attachment creation form.

	lenochod
	Comment 6 • 14 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

	Bruno 'Aqualon' Escherl
	Comment 7 • 14 years ago

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

	Justin Wood (:Callek)
	Comment 8 • 14 years ago

(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]

	lenochod
	Comment 9 • 14 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.

	Bruno 'Aqualon' Escherl
	Comment 10 • 14 years ago

(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.

	lenochod
	Comment 11 • 14 years ago

Aqualon, have I make stacktrace with WinDbg?

	lenochod
	Comment 12 • 14 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.

	(mostly gone) XtC4UaLL [:xtc4uall]
	Comment 13 • 14 years ago

(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?

	Robert Kaiser Assignee
	Comment 14 • 14 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?

	Robert Kaiser Assignee
	Comment 15 • 14 years ago

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. ;-)

	Robert Kaiser Assignee
	Comment 16 • 14 years ago

Pushed as http://hg.mozilla.org/comm-central/rev/4f719059999e

	lenochod
	Comment 17 • 14 years ago

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

It works fine now

	neil@parkwaycc.co.uk
	Comment 18 • 14 years ago

(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.