Bug 91547

Summary: [PATCH] Add basic support for RichMedia annotations
Product: poppler Reporter: Tobias Koenig <tobias.koenig>
Component: generalAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: medium CC: oliver.sander
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: Patch to support RichMedia annotations in poppler core
Patch to support RichMedia annotations in poppler core
Patch to support RichMedia annotations in poppler core
Patch to support RichMedia annotations in poppler core
Patch to support RichMedia annotations in poppler core

Description Tobias Koenig 2015-08-04 06:47:39 UTC
Created attachment 117501 [details]
Patch to support RichMedia annotations in poppler core

The attached patch adds the data structures and parsing code for basic support of RichMedia annotations as described in 'Adobe Supplement to the ISO 32000'.

Since most poppler frontends on Linux cannot support the proper playback of Flash media, this patch retrieves enough information (flash player parameters and filespec to embedded data stream) from the PDF, to allow the front end to extract the actual media (e.g. video or audio) and do the playback by the native media player of the system.
Comment 1 Tobias Koenig 2015-08-04 08:01:09 UTC
Created attachment 117503 [details] [review]
Patch to support RichMedia annotations in poppler core
Comment 2 Tobias Koenig 2015-08-04 09:07:55 UTC
Created attachment 117505 [details] [review]
Patch to support RichMedia annotations in poppler core
Comment 3 Albert Astals Cid 2015-08-10 21:17:08 UTC
Please implement proper getters.
Comment 4 Tobias Koenig 2015-08-11 09:21:54 UTC
Created attachment 117624 [details] [review]
Patch to support RichMedia annotations in poppler core

Made all members private and added proper getter methods.
Comment 5 Albert Astals Cid 2015-08-12 20:20:18 UTC
Do we want code on 
  AnnotRichMedia::Deactivation::Deactivation
and 
  AnnotRichMedia::Activation::Activation
to assign a condition if there "Condition" but doesn't match any of the 3 if? otherwise it ends up being not initialized, no?

For the rest i'll say i haven't look at the spec but seems straigh-forward enough.
Comment 6 Tobias Koenig 2015-08-13 06:42:16 UTC
Created attachment 117663 [details] [review]
Patch to support RichMedia annotations in poppler core

The new patch ensures that the 'condition' member of Activation and Deactivation is always initialized.
Comment 7 Albert Astals Cid 2015-08-13 21:58:18 UTC
Pushed.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.