Ubuntu
cheese package

Take Picture button is enabled outside gtk thread lock

Bug #223172 reported by Gary Lasker
12
Affects Status Importance Assigned to Milestone
Cheese
Fix Released
Medium
cheese (Ubuntu)
Fix Released
Undecided
Unassigned
Hardy
Fix Released
Undecided
Unassigned
Intrepid
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: cheese

Inside the camera setup thread, some UI updates are done using
gdk_threads_enter/leave(). However, the call to enable the Take Picture button
currently occurs *after* gdk_threads_leave(), so it is occurring outside the
gtk thread lock.

It's a very simple fix. I've filed a patch against Cheese upstream SVN trunk and it has been accepted.

Related branches

lp:ubuntu/lucid/cheese
Revision history for this message
Gary Lasker (gary-lasker) wrote :

Here is the patch that was used to address the issue.

Gary Lasker (gary-lasker)
Changed in acton:
assignee: nobody → gary-lasker
importance: Undecided → High
status: New → Fix Committed
Bug Watch Updater (bug-watch-updater)
Changed in cheese:
status: Unknown → Fix Released
Steve Magoun (smagoun)
Changed in acton:
milestone: none → beta2
Gary Lasker (gary-lasker)
Changed in acton:
status: Fix Committed → Fix Released
Revision history for this message
Pedro Villavicencio (pedro) wrote :

fixed upstream now.

Changed in cheese:
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in cheese:
status: New → Fix Committed
Revision history for this message
Gary Lasker (gary-lasker) wrote :

I verified this problem has been fixed in the updated version of Cheese in -proposed. The menus now appear reliably and the UI is consistently responsive. This fix has been verified both on desktop Ubuntu 8.04 running natively on a Macbook Pro and in Hildon mode on an Intel Menlow-based MID device (Jax10).

Revision history for this message
Martin Pitt (pitti) wrote :

Copied hardy-proposed to hardy-updates and intrepid.

Changed in cheese:
status: Fix Committed → Fix Released
status: Fix Committed → Fix Released
Michael Terry (mterry)
tags: added: oem-services
Bug Watch Updater (bug-watch-updater)
Changed in cheese:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.