Patch following

If I understand things correct...

Comment on attachment 760397 [details] [diff] [review] Almost copy-paste patch The 'if (!blank)' check you removed is still needed.

Comment on attachment 760397 [details] [diff] [review] Almost copy-paste patch >+ if (!blank) { >+ b.setAttribute("nodefaultsrc", "true"); >+ } Nit: navigator style is to avoid {}s for a single line of code. >- // Stop the existing about:blank load. Otherwise, if aURI >- // doesn't stop in-progress loads on its own, we'll get into >- // trouble with multiple parallel loads running at once. >- b.stop(); Bug 716108 moved this line to its own block in Firefox which is why bug 878747 was able to simply remove that; the equivalent of the rest of the block still exists in Firefox inside an about blank check.

Hope this time I make proper patch

Comment on attachment 763236 [details] [diff] [review] Almost copy-paste patch v2 >+ if (!blank) { >+ b.setAttribute("nodefaultsrc", "true"); >+ } Lose the {}s please. r=me with that fixed.

Sorry, I saw that nit, but forget about it when remaking patch :(, fixed, carrying r+

Sorry, I can't land this at the moment due to Seamonkey being closed from bug 870207. Callek says that things should be coming back soon and he'll take care of it then.