[totem] Add support for VLC "toolbar" argument



commit c5d86dfbc74024a25528b705efa65af8c78101be
Author: Ivaylo Valkov <ivaylo e-valkov org>
Date:   Tue May 18 17:28:03 2010 +0300

    Add support for VLC "toolbar" argument
    
    With different behaviour if a "vlc" mimetype was requested
    or not.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=618929

 browser-plugin/totemPlugin.cpp |   18 ++++++++++++++----
 1 files changed, 14 insertions(+), 4 deletions(-)
---
diff --git a/browser-plugin/totemPlugin.cpp b/browser-plugin/totemPlugin.cpp
index 5382bb2..d7ec610 100644
--- a/browser-plugin/totemPlugin.cpp
+++ b/browser-plugin/totemPlugin.cpp
@@ -2101,11 +2101,21 @@ totemPlugin::Init (NPMIMEType mimetype,
 
 #endif /* TOTEM_NARROWSPACE_PLUGIN */
 
-/* VLC plugin defaults to have its controller hidden
- * But we don't want to hide it if VLC wasn't explicitely
- * requested */
 #ifdef TOTEM_CONE_PLUGIN
-	if (strstr ((const char *) mimetype, "vlc") != NULL)
+	/* VLC plugin defaults to have its controller hidden.
+	 * But we don't want to hide it if VLC wasn't explicitely requested.
+	 * VLC plugin checks for toolbar argument in vlcplugin.cpp: VlcPlugin::init()
+	 * If the argument is missing or not set to "true" the toolbar is hidden.
+	 * We use a different approach:
+	 * - If there is no toolbar argument or it is not set to "false"
+	 *   the  controller is visible (otherwise it is hidden)
+	 * - If VLC is requested explicitely (via mimetype) and
+	 *   the toolbar argument is not set to "true",
+	 *   the controller is hidden (otherwise it is visible)
+	 */
+	if (!GetBooleanValue (args, "toolbar", true) ||
+	    (!GetBooleanValue (args, "toolbar", false) &&
+	     strstr ((const char *) mimetype, "vlc") != NULL))
 		mControllerHidden = true;
 #endif
 



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]