changing enumerator interface again

widget/src/xpwidgets/nsBaseWidget.cpp

@@ -519,15 +519,14 @@ nsBaseWidget::Enumerator::Last()
 
 
 nsresult
-nsBaseWidget::Enumerator::IsDone(PRBool *aDone)
+nsBaseWidget::Enumerator::IsDone()
 {
-    if (!aDone)
-        return NS_ERROR_NULL_POINTER;
   if ((mCurrentPosition == (mChildren.Count() -1)) || mChildren.Count() <= 0 ){ //empty lists always return done
-        *aDone = PR_TRUE;
+    return NS_OK;
+  }
+  else {
+    return NS_COMFALSE;
   }
-    else
-        *aDone = PR_FALSE;
   return NS_OK;
 }
 
@@ -748,4 +747,3 @@ NS_METHOD nsBaseWidget::Paint(nsIRenderingContext& aRenderingContext,
 }
 
 
-

widget/src/xpwidgets/nsBaseWidget.h

@@ -134,7 +134,7 @@ protected:
       virtual nsresult Next();
       virtual nsresult Prev();
       virtual nsresult CurrentItem(nsISupports **aItem);
-      virtual nsresult IsDone(PRBool *aDone);
+      virtual nsresult IsDone();
 
       void Append(nsIWidget* aWidget);
       void Remove(nsIWidget* aWidget);