After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 598231 - When Chromium rings the bell, metacity quits
When Chromium rings the bell, metacity quits
Status: RESOLVED FIXED
Product: metacity
Classification: Other
Component: general
2.28.x
Other Linux
: Normal normal
: ---
Assigned To: Metacity maintainers list
Metacity maintainers list
: 601907 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-10-13 04:02 UTC by Adam Goode
Modified: 2010-02-08 01:02 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to check if the window has a frame before calling bell_flash_window_frame (593 bytes, patch)
2009-10-14 21:12 UTC, alexisdm59 		committed Details | Review

Description Adam Goode 2009-10-13 04:02:03 UTC 
I am using frame_flash and using the Chromium browser. Chromium doesn't use a decorated window. When it beeps, like when searching, metacity quits with:

metacity:ERROR:core/bell.c:211:bell_flash_window_frame: assertion failed: (window->frame != NULL)
Comment 1 alexisdm59 2009-10-14 21:12:28 UTC 
Created attachment 145453 [details] [review]
Patch to check if the window has a frame before calling bell_flash_window_frame
Comment 2 Owen Taylor 2009-11-16 14:27:22 UTC 
*** Bug 601907 has been marked as a duplicate of this bug. ***
Comment 3 Evan Martin (Chromium) 2010-01-17 18:35:53 UTC 
This is a frequently-reported bug against Chrome.  The same patch to fix it has been written twice.  Who can I bug/bribe to get this in?  :)

http://code.google.com/p/chromium/issues/detail?id=27654
Comment 4 Thomas Thurman 2010-01-17 18:42:21 UTC 
Review of attachment 145453 [details] [review]:

Looks good: committed.
Comment 5 Mark Stosberg 2010-02-08 00:59:26 UTC 
I will note a workaround for affected users who find this bug report before a fix is available for them:

You can use gconf-editor and navigate to here: /apps/metacity/general/visual_bell_type

You could uncheck the "visual bell" option, or change visual_bell_type to fullscreen.
Comment 6 Evan Martin (Chromium) 2010-02-08 01:02:08 UTC 
Due to this and other bugs related to beeping, we removed the bell from Chrome.  It was only added for a11y reasons.  Maybe we'll revisit this once we expect most users have the fixed Metacity.