483532 - When launching w/ downloads window closed, first download appears at top of downloads window

Status: RESOLVED FIXED

(Camino Graveyard :: Downloading, defect)

Description

[Ilya Sherman]

Regression from bug 406103, in which we moved a lot of things from |init| to |awakeFromNib|.  It seems like the method call sequence is now something like first init the downloads window, then add the download, and then call awakeFromNib:, which is when existing downloads are loaded.

STR:
 1) Close downloads window
 2) Relaunch Camino
 3) Initiate a download

Result:
 Download appears at top of list, rather than at the bottom.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Won't block b3, but we should really try hard to have it fixed by then, since it's rather odd.

Comment 2

[Ilya Sherman]

Attachment: Fix

Load model in init rather than in awakeFromNib

Comment 3

[Ilya Sherman]

Attachment: Fix, refreshed

Yar, removes an unrelated code change

Comment 4

[Stuart Morgan]

Comment on attachment 368991 [details] [diff] [review]
Fix, refreshed

r=smorgan

Comment 5

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 368991 [details] [diff] [review]
Fix, refreshed

sr=pink

Comment 6

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Checked in on cvs trunk.