53426 - recent files menu never updates

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P2)

Description

[Kathleen :Brade]

This may be Mac-specific (?).
In the editor, the recent files menu doesn't update.

Steps to reproduce:
  Open a file in Composer (fileA)
  Open another file in Composer (fileB)
  Switch back to window with fileA
  Check Recent Pages menu

I expect that fileB will be listed first in the Recent Pages menu.
Instead, fileB isn't shuffled to the proper location or added to the menu at all 
(if it hadn't been opened before).

Chris--is it possible that the menu isn't rebuilding itself in the mac menu code?  
If that's true, does that mean the bookmarks menu isn't updating itself after 
being edited?  Or mail folders listed in a menu?

Can anyone reproduce this on Linux or Windows?

Comment 1

[Charles Manske]

This was working last time I tested it on Windows (a few days ago)
I know there's been problems just on Mac

Comment 2

[rubydoo123]

tested this using win98 on build 2000092005 comm build and it works correctly.
Asking Sujay to check it out on linux

Comment 3

[Kathleen :Brade]

This appears to be Mac-specific.  I can't reproduce it on my Linux build.  Ryan 
can't reproduce it on his Windows build.

Comment 4

[rubydoo123]

Kathy, can you work on this one? On my mac the list is corrupted and unusable

Comment 5

[saari (gone)]

Should be fixed now...

Comment 6

[Kathleen :Brade]

this is fixed (thanks saari!)
I do get a new assertion in my mozilla debug build:
 ###!!! ASSERTION: Not implemented: '0' file nsMenu.cpp line 477

Comment 7

[sujay]

REopening because I don't see it working on all platforms now..

simply follow the above testcase...

Steps to reproduce:
  Open a file in Composer (fileA)
  Open another file in Composer (fileB)
  Switch back to window with fileA
  Check Recent Pages menu


I do not see File 1 and File 2 in either of the Recent page menus
for these files on ANY platform..

I'm using 9/29 branch build.

Comment 8

[rubydoo123]

commonly used feature, is very visible, fix is low-risk

Kathy, please follow the checkin rules -- get a patch to fix the problems, get 
it super-reviewed, get module owner approval. The reviewer and module owner must 
make an entry in the bug, once all of that is done, remove the NEED INFO in the 
rtm+ block. I will then pop it to pdt for approval.

Comment 9

[Kathleen :Brade]

I can't reproduce this bug on Linux (trunk) or Macintosh (trunk or branch)

Sujay--can you help out with this one?  What builds are people using to see this 
bug?  When did this break?

Everyone--remember, you do *NOT* see the name of the file currently being edited 
in the Recent Files menu (as spec'd).

Comment 10

[Phil Peterson]

PDT agrees [rtm need info] until patch and code reviews are available.

Comment 11

[sujay]

ok I wasn't aware of the spec. its fine....this was the issue.

-remember, you do *NOT* see the name of the file currently being edited 
in the Recent Files menu (as spec'd).

Comment 12

[sujay]

verified in 9/29 build.

Comment 13

[Kathleen :Brade]

I don't think this was ever verified as being a problem on any platform except 
Macintosh so changing platform/OS back.

Thanks Sujay for verifying...