261315 - program name lowercasing should happen in Startup()

Status: RESOLVED FIXED

(Core Graveyard :: X-remote, defect)

Description

[Brian Ryner (not reading)]

currently the program name for xremote is lowercased in AddBrowserInstance, this
doesn't work for non-browser apps like thunderbird.  It should happen just once
in Startup().  This is breaking x-remote mailto: handling for thunderbird.

Comment 1

[Brian Ryner (not reading)]

Attachment: patch

Comment 2

[Brian Ryner (not reading)]

nominating for blocking-1.0, this is a pretty major integration issue.. if you
already have thunderbird running, mailto links and send link don't work.

Comment 3

[Brian Ryner (not reading)]

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

nominating for 1.7 and aviary branch approval

Comment 4

[chris hofmann]

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

a=chofmann for the branches

Comment 5

[Brian Ryner (not reading)]

checked in on trunk and aviary branch.  There's actually nothing to check in on
the 1.7 branch because blizzard's prerequisite patch for bug 244060 never landed
on the 1.7 branch.

Comment 6

[sairuh (rarely reading bugmail)]

not sure if the following is related to this bug --if not, I can file a separate
report.

1. start thunderbird (eg, I've got 2004092705-0.8).
2. start firefox (eg, I've got 2004092709-0.9).
3. in firefox, click either a mailto: link or select File > Send Link.

results: get an alert dialog which says, "The file
resource:///chrome/comm.jar!/content/navigator/navigator.xul cannot be found.
Please check the location and try again."

after I click OK in the alert, tbird crashes with this in the shell:
./run-mozilla.sh: line 451:  1034 Segmentation fault      "$prog" ${1+"$@"}

(no talkback as talkback isn't in today's bits so far.) 

Comment 7

[sairuh (rarely reading bugmail)]

spun off bug 261961 for the issue in comment 6.