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 645975 - video recording makes black-only movies after taking a photo or recording another movie
video recording makes black-only movies after taking a photo or recording an...
Status: RESOLVED FIXED
Product: cheese
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: 2.32
Assigned To: Cheese Maintainer(s)
Cheese Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2011-03-28 16:49 UTC by Raluca-Elena Podiuc
Modified: 2011-04-02 17:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
lib: fix recording videos after previous image/video captures (1.21 KB, patch)
2011-03-29 01:26 UTC, Raluca-Elena Podiuc
accepted-commit_after_freeze Details | Review

Description Raluca-Elena Podiuc 2011-03-28 16:49:04 UTC
Step 1.Open cheese
Step 2.Take a photo
Step 3.Start recording a video
Step 4.Stop recording after a few seconds
Step 5.Open the video in Totem and notice that the timeline is fixed to the end   and stuck there (can't seek in the movie) and the image is completely black although the sound is ok.
Comment 1 Raluca-Elena Podiuc 2011-03-28 20:10:12 UTC
I found a solution, but, at the moment the sound is a bit distorted or unsynchronized. I'll work a bit on it and I'll post the patch in the morning.
Comment 2 Raluca-Elena Podiuc 2011-03-29 01:26:53 UTC
Created attachment 184523 [details] [review]
lib: fix recording videos after previous image/video captures
Comment 3 daniel g. siegel 2011-03-29 19:01:55 UTC
seems ok to me. do you want to request a freeze break and commit the patch?
Comment 4 Raluca-Elena Podiuc 2011-03-29 19:10:10 UTC
are you asking me or some other maintainer?
Comment 5 daniel g. siegel 2011-03-29 19:15:06 UTC
yep. if you don't want to, then i could do it too.
Comment 6 Luciana Fujii 2011-03-29 19:25:47 UTC
Review of attachment 184523 [details] [review]:

Thank you very much for the patch. We are currently on a hard code freeze, but we will commit it after it.
Comment 7 Luciana Fujii 2011-03-29 19:29:27 UTC
Sorry, I didn't see Daniel's comment before. Go ahead and ask for freeze break if you want to.

I think the ideal thing to do would be to fix this at camerabin. Can you provide more information on why this is needed and report it to camerabin also?

Thanks again.
Comment 8 Raluca-Elena Podiuc 2011-03-29 19:41:32 UTC
Freeze request sent. 

I'm not sure why this works, I've tried several variations. Some created very bad interaction between sound/video (lots of cracking), some made the recorded video black too.

GST_STATE_READY's description said that the buffers are flushed or something like that (can't find the docs now). I assumed that having data on the buffer would mess up the transition from IMAGE_MODE to VIDEO_MODE.
Comment 9 Raluca-Elena Podiuc 2011-03-30 09:04:44 UTC
I have received a freeze break permission.
Comment 10 daniel g. siegel 2011-03-31 20:10:57 UTC
please do as the r-t told you, then commit the patch

thanks!
Comment 11 Raluca-Elena Podiuc 2011-04-01 16:30:04 UTC
I submitted bug to gstreamer : 646211
I don't have a gnome git account so I can't commit the patch.
Could you please do that for me?
Comment 12 daniel g. siegel 2011-04-02 17:29:11 UTC
committed. 

thanks a lot!