Added suppressAlarms attribute to oeICal and suppressAlarmsByDefault to oeICalContainer

This is the first step to achieve the goal in bug 257428:
alarm sound / popup box / show preferences on a per-calendar basis

calendar/libxpical/oeICalContainerImpl.cpp

@@ -225,6 +225,7 @@ oeICalContainerImpl::oeICalContainerImpl()
 #endif
 
         m_batchMode = false;
+        m_suppressAlarmsByDefault = false;
 
         NS_NewISupportsArray(getter_AddRefs(m_calendarArray));
         if ( m_calendarArray == nsnull ) {
@@ -332,6 +333,8 @@ oeICalContainerImpl::AddCalendar( const char *server, const char *type ) {
 
     calendar->SetBatchMode( m_batchMode ); //Make sure the current batchmode value is inherited
 
+    calendar->SetSuppressAlarms( m_suppressAlarmsByDefault );
+
     calendar->SetServer( server );
 
     return NS_OK;
@@ -482,6 +485,19 @@ NS_IMETHODIMP oeICalContainerImpl::SetBatchMode(PRBool aBatchMode)
     return NS_OK;
 }
 
+/* attribute boolean suppressAlarmsByDefault; */
+NS_IMETHODIMP oeICalContainerImpl::GetSuppressAlarmsByDefault(PRBool *aRetVal)
+{
+    *aRetVal = m_suppressAlarmsByDefault;
+    return NS_OK;
+}
+
+NS_IMETHODIMP oeICalContainerImpl::SetSuppressAlarmsByDefault(PRBool aNewVal)
+{
+    m_suppressAlarmsByDefault = aNewVal;
+    return NS_OK;
+}
+
 NS_IMETHODIMP oeICalContainerImpl::AddEvent( oeIICalEvent *icalevent, const char *server, char **retid )
 {
 #ifdef ICAL_DEBUG

calendar/libxpical/oeICalContainerImpl.h

@@ -106,6 +106,7 @@ public:
     icaltimetype GetNextEvent( icaltimetype starting );
 private:
     bool m_batchMode;
+    bool m_suppressAlarmsByDefault;
     nsCOMPtr<nsISupportsArray> m_calendarArray;
     nsCOMPtr<nsISupportsArray> m_observerArray;
     nsCOMPtr<nsISupportsArray> m_todoobserverArray;

calendar/libxpical/oeICalImpl.cpp

@@ -353,6 +353,7 @@ oeICalImpl::oeICalImpl()
 #endif
 
         m_batchMode = false;
+        m_suppressAlarms = false;
 
         m_alarmtimer = nsnull;
 
@@ -368,18 +369,7 @@ oeICalImpl::~oeICalImpl()
 #ifdef ICAL_DEBUG
     printf( "oeICalImpl::~oeICalImpl()\n" );
 #endif
-    if( m_alarmtimer  ) {
-        PRUint32 delay = 0;
-        #ifdef NS_INIT_REFCNT //A temporary way of keeping backward compatibility with Mozilla 1.0 source compile
-        delay = m_alarmtimer->GetDelay();
-        #else
-        m_alarmtimer->GetDelay( &delay );
-        #endif
-        if ( delay != 0 )
-            m_alarmtimer->Cancel();
-        m_alarmtimer->Release();
-        m_alarmtimer = nsnull;
-    }
+    CancelAlarmTimer();
 
     NS_RELEASE( m_filter );
 }
@@ -956,6 +946,30 @@ NS_IMETHODIMP oeICalImpl::SetBatchMode(PRBool aBatchMode)
     return NS_OK;
 }
 
+/* attribute boolean suppressAlarms; */
+NS_IMETHODIMP oeICalImpl::GetSuppressAlarms(PRBool *aRetVal)
+{
+    *aRetVal = m_suppressAlarms;
+
+    return NS_OK;
+}
+
+
+NS_IMETHODIMP oeICalImpl::SetSuppressAlarms(PRBool aNewVal)
+{
+    if( m_suppressAlarms ) {
+        if( !aNewVal ) {
+            m_suppressAlarms = PR_FALSE;
+            SetupAlarmManager();
+        }
+    } else {
+        if( aNewVal ) {
+            m_suppressAlarms = PR_TRUE;
+            CancelAlarmTimer();
+        }
+    }
+    return NS_OK;
+}
 
 /**
 *
@@ -1804,7 +1818,8 @@ void oeICalImpl::SetupAlarmManager() {
 #ifdef ICAL_DEBUG
     printf( "oeICalImpl::SetupAlarmManager()\n" );
 #endif
-
+    if( m_suppressAlarms )
+        return;
     if( m_batchMode )
     {
         #ifdef ICAL_DEBUG
@@ -1973,18 +1988,8 @@ void oeICalImpl::SetupAlarmManager() {
 
     lastcheck = now;
 
-    if( m_alarmtimer  ) {
-        PRUint32 delay = 0;
-        #ifdef NS_INIT_REFCNT //A temporary way of keeping backward compatibility with Mozilla 1.0 source compile
-        delay = m_alarmtimer->GetDelay();
-        #else
-        m_alarmtimer->GetDelay( &delay );
-        #endif
-        if ( delay != 0 )
-            m_alarmtimer->Cancel();
-        m_alarmtimer->Release();
-        m_alarmtimer = nsnull;
-    }
+    CancelAlarmTimer(); //cancel the current alarm timer if any
+
     if( !icaltime_is_null_time( nextalarm ) ) {
         #ifdef ICAL_DEBUG
         printf( "NEXT ALARM IS: %s\n", icaltime_as_ical_string( nextalarm ) );
@@ -2005,6 +2010,21 @@ void oeICalImpl::SetupAlarmManager() {
     }
 }
 
+void oeICalImpl::CancelAlarmTimer() {
+    if( m_alarmtimer  ) {
+        PRUint32 delay = 0;
+        #ifdef NS_INIT_REFCNT //A temporary way of keeping backward compatibility with Mozilla 1.0 source compile
+        delay = m_alarmtimer->GetDelay();
+        #else
+        m_alarmtimer->GetDelay( &delay );
+        #endif
+        if ( delay != 0 )
+            m_alarmtimer->Cancel();
+        m_alarmtimer->Release();
+        m_alarmtimer = nsnull;
+    }
+}
+
 /**
 *
 *   AddTodo

calendar/libxpical/oeICalImpl.h

@@ -246,6 +246,7 @@ private:
     nsCOMPtr<nsISupportsArray> m_observerlist;
     nsCOMPtr<nsISupportsArray> m_todoobserverlist;
     bool m_batchMode;
+    bool m_suppressAlarms;
     EventList m_eventlist;
     TodoList m_todolist;
     nsITimer *m_alarmtimer;
@@ -254,6 +255,7 @@ private:
     bool SatisfiesFilter( oeIICalTodo *comp );
     void ChopAndAddEventToEnum( struct icaltimetype startdate,
                                         nsISimpleEnumerator **eventlist, oeICalEventImpl* event, bool isallday, bool isbeginning );
+    void CancelAlarmTimer();
 };
 
 #endif

calendar/libxpical/oeIICal.idl

@@ -219,6 +219,7 @@ interface oeIICal : nsISupports
     attribute boolean batchMode;
     readonly attribute oeIICalTodo filter;
     attribute string server;
+    attribute boolean suppressAlarms;
 
     void addObserver( in oeIICalObserver observer );
     void removeObserver( in oeIICalObserver observer );
@@ -256,6 +257,7 @@ interface oeIICal : nsISupports
 interface oeIICalContainer : nsISupports
 {
     attribute boolean batchMode;
+    attribute boolean suppressAlarmsByDefault;
     readonly attribute oeIICalTodo filter;