Bug 353171

Summary: Loop playback doesn't work for certain videos embedded in PDF
Product: [Applications] okular Reporter: Arseniy Lartsev <receive-spam>
Component: generalAssignee: Okular developers <okular-devel>
Status: RESOLVED FIXED    
Severity: normal CC: aacid
Priority: NOR    
Version: 0.23.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
URL: https://git.reviewboard.kde.org/r/125341/
Latest Commit: http://commits.kde.org/okular/e9e5cb6317d2f46334dc624663b2999c1351f19c Version Fixed In:
Attachments: Test case
Source for the pdf

Description Arseniy Lartsev 2015-09-25 12:22:17 UTC 
Videos embedded by Latex movie15 package will always play once, regardless of the loop playback settings.

Patch: https://git.reviewboard.kde.org/r/125341/

Reproducible: Always
Comment 1 Arseniy Lartsev 2015-09-25 12:26:18 UTC 
Created attachment 94717 [details]
Test case
Comment 2 Arseniy Lartsev 2015-09-25 12:26:48 UTC 
Created attachment 94718 [details]
Source for the pdf
Comment 3 Albert Astals Cid 2015-09-26 15:30:23 UTC 
Git commit e9e5cb6317d2f46334dc624663b2999c1351f19c by Albert Astals Cid, on behalf of Arseniy Lartsev.
Committed on 26/09/2015 at 15:30.
Pushed by aacid into branch 'master'.

Repeat videos in PDFs automatically when needed

REVIEW: 125341

M  +13   -1    core/movie.cpp
M  +14   -2    core/movie.h
M  +6    -1    generators/poppler/generator_pdf.cpp
M  +16   -7    ui/videowidget.cpp

http://commits.kde.org/okular/e9e5cb6317d2f46334dc624663b2999c1351f19c
Comment 4 Paul 2015-10-24 17:46:24 UTC 
Don't know if anyone will see this as its resolved, but don't know where better to post this. it appears this commit was applied to an outdated version of generator_pdf.cpp.  There is one occurrence of 'Okular::Movie::playOnce' introduced in a recent commit (and behind an #ifdef HAVE_POPPLER_0_36), but playOnce was removed from the enum in this commit.
Comment 5 Albert Astals Cid 2015-10-26 18:41:30 UTC 
Paul, you're probably speaking about the unsupported frameworks branch, right?
Comment 6 Paul 2015-10-26 18:43:44 UTC 
Yes, apologies for forgetting to mention that.
Comment 7 Albert Astals Cid 2015-10-26 18:57:50 UTC 
Alex Merry fixed it this morning
Comment 8 Paul 2015-10-26 18:58:27 UTC 
Much appreciated :)