Attachment 67756 Details for Bug 55378 – Patch to extend Poppler::LinkRendition in Qt4 frontend

Patch to extend Poppler::LinkRendition in Qt4 frontend

poppler_qt4_linkrendition.patch (text/plain), 6.68 KB, created by Tobias Koenig on 2012-09-27 08:30:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tobias Koenig

Created: 2012-09-27 08:30:36 UTC

Size: 6.68 KB

patch

obsolete

>diff --git a/poppler/Link.cc b/poppler/Link.cc
>index b90b1c1..718fdbd 100644
>--- a/poppler/Link.cc
>+++ b/poppler/Link.cc
>@@ -712,8 +712,8 @@ LinkRendition::LinkRendition(Object *obj) {
>         js = new GooString(tmp.getString());
>       } else if (tmp.isStream()) {
>         Stream *stream = tmp.getStream();
>-	js = new GooString();
>-	stream->fillGooString(js);
>+        js = new GooString();
>+        stream->fillGooString(js);
>       } else {
>         error(errSyntaxWarning, -1, "Invalid Rendition Action: JS not string or stream");
>       }
>@@ -730,15 +730,15 @@ LinkRendition::LinkRendition(Object *obj) {
>         // retrieve rendition object
>         if (obj->dictLookup("R", &renditionObj)->isDict()) {
>           media = new MediaRendition(&renditionObj);
>-	} else if (operation == 0 || operation == 4) {
>+        } else if (operation == 0 || operation == 4) {
>           error(errSyntaxWarning, -1, "Invalid Rendition Action: no R field with op = {0:d}", operation);
>-	  renditionObj.free();
>-	}
>+          renditionObj.free();
>+        }
> 
>-	if (!obj->dictLookupNF("AN", &screenRef)->isRef() && operation >= 0 && operation <= 4) {
>-	  error(errSyntaxWarning, -1, "Invalid Rendition Action: no AN field with op = {0:d}", operation);
>-	  screenRef.free();
>-	}
>+        if (!obj->dictLookupNF("AN", &screenRef)->isRef() && operation >= 0 && operation <= 4) {
>+          error(errSyntaxWarning, -1, "Invalid Rendition Action: no AN field with op = {0:d}", operation);
>+          screenRef.free();
>+        }
>       }
>     } else if (!js) {
>       error(errSyntaxWarning, -1, "Invalid Rendition action: no OP or JS field defined");
>diff --git a/poppler/Link.h b/poppler/Link.h
>index 8e2df24..6a5c25c 100644
>--- a/poppler/Link.h
>+++ b/poppler/Link.h
>@@ -51,7 +51,7 @@ enum LinkActionKind {
>   actionURI,			// URI
>   actionNamed,			// named action
>   actionMovie,			// movie action
>-  actionRendition,
>+  actionRendition,		// rendition action
>   actionSound,			// sound action
>   actionJavaScript,		// JavaScript action
>   actionOCGState,               // Set-OCG-State action
>diff --git a/qt4/src/poppler-link.cc b/qt4/src/poppler-link.cc
>index a4bc55b..329419c 100644
>--- a/qt4/src/poppler-link.cc
>+++ b/qt4/src/poppler-link.cc
>@@ -168,16 +168,41 @@ class LinkSoundPrivate : public LinkPrivate
> class LinkRenditionPrivate : public LinkPrivate
> {
> 	public:
>-		LinkRenditionPrivate( const QRectF &area, ::MediaRendition *rendition );
>+		LinkRenditionPrivate( const QRectF &area, ::MediaRendition *rendition, int operation, const QString &script );
> 		~LinkRenditionPrivate();
> 
> 		MediaRendition *rendition;
>+		LinkRendition::RenditionAction action;
>+		QString script;
> };
> 
>-	LinkRenditionPrivate::LinkRenditionPrivate( const QRectF &area, ::MediaRendition *r )
>+	LinkRenditionPrivate::LinkRenditionPrivate( const QRectF &area, ::MediaRendition *r, int operation, const QString &javaScript )
> 		: LinkPrivate( area )
>-		, rendition( new MediaRendition( r ) )
>+		, rendition( r ? new MediaRendition( r ) : 0 )
>+		, action( LinkRendition::PlayRendition )
>+		, script( javaScript )
> 	{
>+		switch ( operation )
>+		{
>+			case -1:
>+				action = LinkRendition::NoRendition;
>+				break;
>+			case 0:
>+				action = LinkRendition::PlayRendition;
>+				break;
>+			case 1:
>+				action = LinkRendition::StopRendition;
>+				break;
>+			case 2:
>+				action = LinkRendition::PauseRendition;
>+				break;
>+			case 3:
>+				action = LinkRendition::ResumeRendition;
>+				break;
>+			case 4:
>+				action = LinkRendition::PlayRendition;
>+				break;
>+		}
> 	}
> 
> 	LinkRenditionPrivate::~LinkRenditionPrivate()
>@@ -578,26 +603,38 @@ class LinkMoviePrivate : public LinkPrivate
> 	}
> 
> 	// LinkRendition
>-	LinkRendition::LinkRendition( const QRectF &linkArea, ::MediaRendition *rendition )
>-		: Link( *new LinkRenditionPrivate( linkArea, rendition ) )
>+	LinkRendition::LinkRendition( const QRectF &linkArea, ::MediaRendition *rendition, int operation, const QString &script )
>+		: Link( *new LinkRenditionPrivate( linkArea, rendition, operation, script ) )
> 	{
> 	}
>-	
>+
> 	LinkRendition::~LinkRendition()
> 	{
> 	}
>-	
>+
> 	Link::LinkType LinkRendition::linkType() const
> 	{
> 		return Rendition;
> 	}
>-	
>+
> 	MediaRendition * LinkRendition::rendition() const
> 	{
> 		Q_D( const LinkRendition );
> 		return d->rendition;
> 	}
> 
>+	LinkRendition::RenditionAction LinkRendition::action() const
>+	{
>+		Q_D( const LinkRendition );
>+		return d->action;
>+	}
>+
>+	QString LinkRendition::script() const
>+	{
>+		Q_D( const LinkRendition );
>+		return d->script;
>+	}
>+
> 	// LinkJavaScript
> 	LinkJavaScript::LinkJavaScript( const QRectF &linkArea, const QString &js )
> 		: Link( *new LinkJavaScriptPrivate( linkArea ) )
>diff --git a/qt4/src/poppler-link.h b/qt4/src/poppler-link.h
>index a2ef2d3..a07d7ab 100644
>--- a/qt4/src/poppler-link.h
>+++ b/qt4/src/poppler-link.h
>@@ -455,12 +455,28 @@ class POPPLER_QT4_EXPORT LinkRendition : public Link
> {
> 	public:
> 		/**
>-		 * Create a new media rendition link.
>+		 * Describes the possible rendition actions.
>+		 *
>+		 * \since 0.22
>+		 */
>+		enum RenditionAction {
>+			NoRendition,
>+			PlayRendition,
>+			StopRendition,
>+			PauseRendition,
>+			ResumeRendition
>+		};
>+
>+		/**
>+		 * Create a new rendition link.
> 		 *
> 		 * \param linkArea the active area of the link
>-		 * \param rendition 
>+		 * \param rendition the media rendition object
>+		 * \param operation the numeric operation (action)
>+		 * \param script the java script code
> 		 */
>-		LinkRendition( const QRectF &linkArea, ::MediaRendition *rendition );
>+		LinkRendition( const QRectF &linkArea, ::MediaRendition *rendition, int operation, const QString &script );
>+
> 		/**
> 		 * Destructor.
> 		 */
>@@ -469,10 +485,24 @@ class POPPLER_QT4_EXPORT LinkRendition : public Link
> 		LinkType linkType() const;
> 
> 		/**
>-		 * 
>+		 * Returns the media rendition object if the redition provides one, @c 0 otherwise
> 		 */
> 		MediaRendition *rendition() const;
> 
>+		/**
>+		 * Returns the action that should be executed if a rendition object is provided.
>+		 *
>+		 * \since 0.22
>+		 */
>+		RenditionAction action() const;
>+
>+		/**
>+		 * The JS code that shall be executed or an empty string.
>+		 *
>+		 * \since 0.22
>+		 */
>+		QString script() const;
>+
> 	private:
> 		Q_DECLARE_PRIVATE( LinkRendition )
> 		Q_DISABLE_COPY( LinkRendition )
>diff --git a/qt4/src/poppler-page.cc b/qt4/src/poppler-page.cc
>index 349a1bb..ebd7d1c 100644
>--- a/qt4/src/poppler-page.cc
>+++ b/qt4/src/poppler-page.cc
>@@ -197,7 +197,7 @@ Link* PageData::convertLinkActionToLink(::LinkAction * a, DocumentData *parentDo
>     case actionRendition:
>     {
>       ::LinkRendition *lrn = (::LinkRendition *)a;
>-      popplerLink = new LinkRendition( linkArea, lrn->getMedia() );
>+      popplerLink = new LinkRendition( linkArea, lrn->getMedia(), lrn->getOperation(), UnicodeParsedString( lrn->getScript() ) );
>     }
>     break;
>

Actions: View

Attachments on bug 55378: 67756 | 67766 | 68242

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.