Closed
Bug 399161
Opened 18 years ago
Closed 16 years ago
only call gnome_program_init if there is no GnomeProgram yet
Categories
(Toolkit :: Startup and Profile System, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.3a1
People
(Reporter: chpe, Assigned: chpe)
References
Details
Attachments
(1 file, 1 obsolete file)
|
2.03 KB,
patch
|
Details | Diff | Splinter Review |
nsNativeAppSupportsUnix::Start always calls gnome_program_init. However, this is not desirable since if gnome_program_init was called before XRE_main, there already exists a GnomeProgram instance; and calling it again will override the application name to "Gecko". (See also bug 221012.)
| Assignee | ||
Comment 1•18 years ago
|
||
Attachment #284147 -
Flags: review?(benjamin)
Comment 2•18 years ago
|
||
Any chance we could just replace the gnome_program_init stuff with xlib code like Havoc suggested? He said there was some code in Metacity you might be able to copy.
http://mail.gnome.org/archives/desktop-devel-list/2007-July/msg00060.html
Comment 3•18 years ago
|
||
Metacity's session management code lives here:
http://svn.gnome.org/viewvc/metacity/trunk/src/session.c?view=markup
I don't really know any of this code so I can't make a patch out of it :-(
Updated•18 years ago
|
Attachment #284147 -
Flags: review?(benjamin) → review+
| Assignee | ||
Comment 4•18 years ago
|
||
(In reply to comment #2)
> Any chance we could just replace the gnome_program_init stuff with xlib code
> like Havoc suggested? He said there was some code in Metacity you might be able
> to copy.
Please file a separate bug for that; this one is strictly for fixing the use of gnome_program_init.
Component: XRE Startup → Startup and Profile System
QA Contact: xre.startup → startup
Updated•16 years ago
|
Assignee: nobody → chpe
Comment 5•16 years ago
|
||
I unbitrotted chpe's patch and landed it.
http://hg.mozilla.org/mozilla-central/rev/57ff981f69f6
Attachment #284147 -
Attachment is obsolete: true
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
You need to log in
before you can comment on or make changes to this bug.
Description
•