<br><br><div class="gmail_quote">2011/6/1 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Wed, May 25, 2011 at 10:33:10PM +0700, Sergey Radionov wrote :<br>
> Reformated version<br>
<br>
<br>
> + š š š šVlcPlugin* p_plugin = getPrivate<VlcPlugin>();<br>
> +<br>
> š š š š šswitch( index )<br>
> š š š š š{<br>
> š š š š š š šcase ID_root_audio:<br>
> @@ -144,6 +148,44 @@ LibvlcRootNPObject::getProperty(int index, NPVariant &result)<br>
> š š š š š š š š šreturn INVOKERESULT_NO_ERROR;<br>
> š š š š š š šcase ID_root_VersionInfo:<br>
> š š š š š š š š šreturn invokeResultString(libvlc_get_version(),result);<br>
> + š š š š š šcase ID_root_NewMessageFlag:<br>
> + š š š š š š{<br>
> + š š š š š š š šbool val = p_plugin->getNewMessageFlag();<br>
> + š š š š š š š šBOOLEAN_TO_NPVARIANT(val, result);<br>
> + š š š š š š š šreturn INVOKERESULT_NO_ERROR;<br>
> + š š š š š š}<br>
> + š š š š š šdefault:<br>
> + š š š š š š š š;<br>
> + š š š š}<br>
> + š š}<br>
> + š šreturn INVOKERESULT_GENERIC_ERROR;<br>
> +}<br>
> +RuntimeNPObject::InvokeResult<br>
> +LibvlcRootNPObject::setProperty(int index, const NPVariant &value)<br>
> +{<br>
> + š š/* is plugin still running */<br>
> + š šif( isPluginRunning() )<br>
> + š š{<br>
> + š š š šVlcPlugin* p_plugin = getPrivate<VlcPlugin>();<br>
> +<br>
> + š š š š/*<br>
> + š š š šlibvlc_media_player_t *p_md = p_plugin->getMD();<br>
> + š š š šif( !p_md )<br>
> + š š š š š šRETURN_ON_ERROR;<br>
> + š š š š*/<br>
> + š š š šswitch( index )<br>
> + š š š š{<br>
> + š š š š š šcase ID_root_NewMessageFlag:<br>
> + š š š š š š{<br>
> + š š š š š š š šif( ! NPVARIANT_IS_BOOLEAN(value) )<br>
> + š š š š š š š š{<br>
> + š š š š š š š š š šreturn INVOKERESULT_INVALID_VALUE;<br>
> + š š š š š š š š}<br>
> +<br>
> + š š š š š š š šbool val = NPVARIANT_TO_BOOLEAN(value);<br>
> + š š š š š š š šp_plugin->setNewMessageFlag(val);<br>
> + š š š š š š š šreturn INVOKERESULT_NO_ERROR;<br>
> + š š š š š š}<br>
> š š š š š š šdefault:<br>
> š š š š š š š š š;<br>
<br>
> diff --git a/projects/mozilla/control/npolibvlc.h b/projects/mozilla/control/npolibvlc.h<br>
> index c6d8694..70477ee 100644<br>
> --- a/projects/mozilla/control/npolibvlc.h<br>
> +++ b/projects/mozilla/control/npolibvlc.h<br>
> @@ -46,6 +46,7 @@ protected:<br>
> š š šstatic const NPUTF8 * const propertyNames[];<br>
><br>
> š š šInvokeResult getProperty(int index, NPVariant &result);<br>
> + š šInvokeResult setProperty(int index, const NPVariant &value);<br>
><br>
> š š šstatic const int methodCount;<br>
> š š šstatic const NPUTF8 * const methodNames[];<br>
<br>
All this should be in a separate patch, :D<br>
<br>
<br>
<br>
> +typedef VlcPlugin VLCPlugin;<br>
Why?<br></blockquote><div>To make maximum amount of code identical in ActiveX and Mozilla versions. It helps synchronize changes, <br>cause to match amount of code is identical (and logic is <span id="result_box" class="short_text" lang="en"><span title="îÁÖÍÉÔÅ, ÞÔÏÂÙ Õ×ÉÄÅÔØ ÁÌØÔÅÒÎÁÔÉ×ÎÙÊ ÐÅÒÅ×ÏÄ" class="hps">absolutely identical</span></span>).<br>

š
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Exactly the same remarks on the FS controller than on the ActiveX part.<br></blockquote><div>Ok, Accepted.<br>š<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

How much of the code is identical?<br></blockquote><div>See comment above.<br><br>May be I make fullscreen part <span id="result_box" class="short_text" lang="en"><span title="îÁÖÍÉÔÅ, ÞÔÏÂÙ Õ×ÉÄÅÔØ ÁÌØÔÅÒÎÁÔÉ×ÎÙÊ ÐÅÒÅ×ÏÄ" class="hps">self-sufficient</span></span>, so anybody could use it in any project usig libvlc... I'll think <span id="result_box" class="short_text" lang="en"><span title="îÁÖÍÉÔÅ, ÞÔÏÂÙ Õ×ÉÄÅÔØ ÁÌØÔÅÒÎÁÔÉ×ÎÙÊ ÐÅÒÅ×ÏÄ" class="hps">whether it is possible or not... </span></span><br>
<br></div><div></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Fullscreen part should be ok.<br>
<br>
Best Regards,<br>
<font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - +33 672 704 734<br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></blockquote></div><br>