Merge lp:~sylvain-pineau/checkbox/story482.7_screenshot_fullscreen_video into lp:checkbox
Proposed by
Sylvain Pineau
Status: | Merged |
---|---|
Merged at revision: | 1302 |
Proposed branch: | lp:~sylvain-pineau/checkbox/story482.7_screenshot_fullscreen_video |
Merge into: | lp:checkbox |
Diff against target: |
176 lines (+97/-15) (has conflicts) 4 files modified
debian/changelog (+10/-0) debian/control (+1/-1) jobs/graphics.txt.in (+21/-0) scripts/gst_pipeline_test (+65/-14) Text conflict in debian/changelog |
To merge this branch: | bzr merge lp:~sylvain-pineau/checkbox/story482.7_screenshot_fullscreen_video |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Javier Collado (community) | Approve | ||
Marc Tardif (community) | Approve | ||
Jeff Lane | Needs Fixing | ||
Review via email: mp+95575@code.launchpad.net |
Description of the change
This proposal comes with a new gst_pipeline_test script which support fullscreen video playback by forcing the video sink element to use a specific Gtk Window identifier for rendering.
A new features ensures that the video playback will loop until the given timeout is reached by seeking to the first frame on EOS events.
A new job takes care of performing a still capture during a fullscreen video playback.
To post a comment you must log in.
I'm not comfortable with hard coding /dev/video1 into the job command like that.
On the system I tested with, I somehow ended up with three video devices:
bladernr@klaatu:~$ find /dev -name video*
/dev/video1
/dev/video0
bladernr@klaatu:~$ find /dev -name video*
/dev/video2
/dev/video1
/dev/video0
And in my system, the USB webcam I just plugged in is video2, not video1.
Also, some systems may not already have a built in webcam... for example, my Thinkpad x201:
bladernr@ GarbageScow: ~/Downloads$ find /dev -name video* GarbageScow: ~/Downloads$
bladernr@
and after plugging in the webcam:
bladernr@ GarbageScow: ~/Downloads$ find /dev -name video* GarbageScow: ~/Downloads$
/dev/video0
bladernr@