390592 - Firefox fails to load all homepages if one of them is a no more existing local dir

Status: VERIFIED DUPLICATE of bug 356571

(Firefox :: Tabbed Browser, defect)

Description

[Ria Klaassen (not reading all bugmail)]

Steps to reproduce:

- New profile, start Firefox
- Create a folder on your desktop and call it Downloads
- In Options, set the default download folder to Downloads
- Drag Downloads from your local disk into the open browser window
- Load 3 other pages as new tabs
- Set all 4 tabs as homepages
- Rename folder Downloads
- Restart the browser

Result: only one message "File not found" but Firefox fails to load the other 3 pages
Expected result: all 4 tabs should be loaded.

Regression between 2006-01-20 13 and 2006-01-20 21: http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=2006-01-20+12%3A00&maxdate=2006-01-20+22%3A00

Could be caused by bug 308396. The bug is also present on branch.
Sorry for the complicated Steps to reproduce, but they all all required.

Comment 1

[Marco Bonardo [:mak]]

Attachment: add try catch in loadTabs

gBrowser.loadTabs(aURIString.split("|"), false, true); sends an exception  on load of the first uri, that is catched and it stop. 
Before it was using a for cycle with gBrowser.addTab(urls[i]); and catching on each iteration
So adding try catch inside loadTabs does the trick

Comment 2

[Marco Bonardo [:mak]]

don't know who is the right reviewer for tabbed brower, mano?
Also, this needs check-in if approved

Comment 3

[Marco Bonardo [:mak]]

editing title, this happens with every non existing local dir
temporary taking bug waiting for review (or something)

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 274901 [details] [diff] [review]
add try catch in loadTabs

Please feel free to move this patch over to the other bug.

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified dup