[Bug 1160569] Re: firefox prints warning to stdout when started from terminal

[Bug Watch Updater]

Launchpad has imported 11 comments from the remote bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=672671.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2011-07-19T22:59:20+00:00 Hiikezoe wrote:

Created attachment 546933
A patch

The following message is spewed out on startup of firefox.

GLib-CRITICAL **: g_slice_set_config: assertion `sys_page_size == 0'
failed


So out jemalloc is less effective.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/0

------------------------------------------------------------------------
On 2011-07-19T23:00:13+00:00 Hiikezoe wrote:

(In reply to comment #0)

> So out jemalloc is less effective.

*our* jemalloc, I meant.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/1

------------------------------------------------------------------------
On 2011-07-20T03:22:06+00:00 Ginn-chen-r wrote:

You can just move the line
static GQuark sQuark_gecko_acc_obj = g_quark_from_static_string("GeckoAccObj");
inside toplevel_event_watcher().

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/2

------------------------------------------------------------------------
On 2011-07-20T03:46:25+00:00 Hiikezoe wrote:

nsApplicationAccessibleWrap::Init() is invoked just once?

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/3

------------------------------------------------------------------------
On 2011-07-20T05:24:47+00:00 Trevor Saunders wrote:

Comment on attachment 546933
A patch

STEALING THE REVIEW, PER REQUEST IN 669130 ON THAT PATCH.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/4

------------------------------------------------------------------------
On 2011-07-20T05:27:45+00:00 Trevor Saunders wrote:

(In reply to comment #2)
> You can just move the line
> static GQuark sQuark_gecko_acc_obj =
> g_quark_from_static_string("GeckoAccObj");
> inside toplevel_event_watcher().

are you absolutely sure that will cause it to be initialized on the
first call, not as a global ctor?

(In reply to comment #3)
> nsApplicationAccessibleWrap::Init() is invoked just once?

it should be although I'm not sure why that would matter, personally I
like the variable being local to the one function we use it in.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/5

------------------------------------------------------------------------
On 2011-07-20T13:34:11+00:00 Ginn-chen-r wrote:

(In reply to comment #5)
> (In reply to comment #2)
> > You can just move the line
> > static GQuark sQuark_gecko_acc_obj =
> > g_quark_from_static_string("GeckoAccObj");
> > inside toplevel_event_watcher().
> 
> are you absolutely sure that will cause it to be initialized on the first
> call, not as a global ctor?
> 

Absolutely sure.

But although it will save a couple of source code lines, the final
binary will be several bytes larger than the patch here, because the
compiler has to do the job for us.

I'm OK with either approach.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/6

------------------------------------------------------------------------
On 2011-07-21T21:36:58+00:00 Bzbarsky wrote:

http://hg.mozilla.org/integration/mozilla-inbound/rev/b0599f36be18

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/7

------------------------------------------------------------------------
On 2011-07-22T21:10:37+00:00 Ehsan-mozilla wrote:

http://hg.mozilla.org/mozilla-central/rev/b0599f36be18

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/8

------------------------------------------------------------------------
On 2013-04-07T22:09:34+00:00 Bugzilla-obra wrote:

This bug appears to have reemerged when firefox runs under glib 2.36

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/13

------------------------------------------------------------------------
On 2013-04-08T13:43:23+00:00 Ginn-chen-r wrote:

(In reply to Fredrik from comment #9)
> This bug appears to have reemerged when firefox runs under glib 2.36

Please file a new bug and it will be great if you can provide crash
stack.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1160569/comments/14


** Changed in: firefox
       Status: Unknown => Fix Released

** Changed in: firefox
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla Firefox.
https://bugs.launchpad.net/bugs/1160569

Title:
  firefox prints warning to stdout when started from terminal

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/1160569/+subscriptions