[xiph-commits] r15074 - trunk/oggPusher/firefox-extension/chrome/content

[nithin at svn.xiph.org]

Author: nithin
Date: 2008-06-26 17:54:19 -0700 (Thu, 26 Jun 2008)
New Revision: 15074

Modified:
   trunk/oggPusher/firefox-extension/chrome/content/op_Window.xul
   trunk/oggPusher/firefox-extension/chrome/content/op_browser.js
   trunk/oggPusher/firefox-extension/chrome/content/op_browser.xul
Log:
resolving conflicts with firefox functionalities

Modified: trunk/oggPusher/firefox-extension/chrome/content/op_Window.xul
===================================================================
--- trunk/oggPusher/firefox-extension/chrome/content/op_Window.xul	2008-06-27 00:30:01 UTC (rev 15073)
+++ trunk/oggPusher/firefox-extension/chrome/content/op_Window.xul	2008-06-27 00:54:19 UTC (rev 15074)
@@ -5,7 +5,6 @@
     id="op-window"
     title="oggPusher"
     orient="horizontal"
-    onclose="onClose()"
     xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 <!-- Other elements go here --> 
 

Modified: trunk/oggPusher/firefox-extension/chrome/content/op_browser.js
===================================================================
--- trunk/oggPusher/firefox-extension/chrome/content/op_browser.js	2008-06-27 00:30:01 UTC (rev 15073)
+++ trunk/oggPusher/firefox-extension/chrome/content/op_browser.js	2008-06-27 00:54:19 UTC (rev 15074)
@@ -1,9 +1,7 @@
-function openDialog()
-{
-   //alert("hello"); 
-   var object = window.open("chrome://oggPusher/content/op_Window.xul",
+var oggPusherCommon={
+	openDialog:function()
+	{
+   		 window.open("chrome://oggPusher/content/op_Window.xul",
                       "oggPusher", "chrome,centerscreen,resizable=yes,toolbar=no,menubar=no");
-	if(object==null)
-	{ alert("inside the if");	}
+	}
 }
-

Modified: trunk/oggPusher/firefox-extension/chrome/content/op_browser.xul
===================================================================
--- trunk/oggPusher/firefox-extension/chrome/content/op_browser.xul	2008-06-27 00:30:01 UTC (rev 15073)
+++ trunk/oggPusher/firefox-extension/chrome/content/op_browser.xul	2008-06-27 00:54:19 UTC (rev 15074)
@@ -13,7 +13,7 @@
 	      class="menuitem-iconic"  
               insertafter="javascriptConsole,devToolsSeparator"
               label="oggPusher"
-	      oncommand="openDialog();"/>
+	      oncommand="oggPusherCommon.openDialog();"/>
   </menupopup>
 	
   <toolbarpalette id="BrowserToolbarPalette">             
@@ -21,7 +21,7 @@
 		class="toolbarbutton-1"  
 		label="oggPusher" 
 		tooltiptext="Open OggPusher"
-		oncommand="openDialog();"/>            
+		oncommand="oggPusherCommon.openDialog();"/>            
   </toolbarpalette>
 
 
@@ -30,7 +30,7 @@
   	<statusbarpanel id="op_status" src="chrome://oggPusher/content/images/xiph.ico" 
               tooltiptext="open oggPusher" 
               class="statusbarpanel-iconic"
-	      oncommand="openDialog();"/>
+	      oncommand="oggPusherCommon.openDialog();"/>
   </statusbar>