108089 - clicking on a desktop link not loading link in current running browser

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Scott MacGregor]

I'm using: 2001103003 build. mozilla is set up as my default browser on win2k.
If the browser is currently running and I click on an http url from another app,
I get the profile manager and it looks like another instance of the app starts
up. If I use the 094 branch, we load the url directly in the browser instance
which is currently running.

I wonder if we are no longer properly detecting the running instance so DDE
thinks we need to start up the app again?

Comment 1

[Bill Law]

worksforme

Do you see a splash screen when you click on the html file?  If not, then we're
detecting the already running process.

Are you using Quick Launch?  I think Conrad may have been doing something with
multi-profile support under Quick Launch and it may be that something is broken
there.

Comment 2

[Scott MacGregor]

No, I'm not using quick launch. My current process is still running. If I choose
the same profile as the one I'm currently running it then creates a new browser
window and there is still only one moizlla process. I haven't tried picking a
different profile yet to see if we attempt to really switch profiles.

Good call on the splash screen. I'm not getting the splash screen just the
profile manager so your right we are detecting the currently running process.

Comment 3

[Bill Law]

It must have something to do with multi-profile support for QuickLaunch, which
is the reason the profile-manager code has changed recently, I think.

Re-assigning to Conrad.

BTW, I created another profile and now I see the problem.

Comment 4

[Conrad Carlen (not reading bugmail)]

While introduced by multiple profile support (my fault), it's not a profile back
end problem. I think this is something in nsNativeAppSupportWin. Component
should stay XPApps.

Comment 5

[Conrad Carlen (not reading bugmail)]

Attachment: patch

Comment 6

[Conrad Carlen (not reading bugmail)]

The old logic using mDidProfileStartup was wrong. Comments are in the patch.

Comment 7

[Scott MacGregor]

Comment on attachment 56878 [details] [diff] [review]
patch

sr=mscott in case you are in need of a sr for this change =).

Comment 8

[Conrad Carlen (not reading bugmail)]

Bill, can you r= the patch?

Comment 9

[Bill Law]

Comment on attachment 56878 [details] [diff] [review]
patch

r=law

Comment 10

[Bill Law]

*** Bug 110492 has been marked as a duplicate of this bug. ***

Comment 11

[Conrad Carlen (not reading bugmail)]

*** Bug 101371 has been marked as a duplicate of this bug. ***

Comment 12

[Conrad Carlen (not reading bugmail)]

Fix checked in.

Comment 13

[Grace Bush]

verified on mozilla build 2001121403