minor cocoa widget cleanups, no bug. spelling, error return value.

widget/src/cocoa/nsIChangeManager.idl

@@ -44,6 +44,8 @@
 interface nsIContent;
 interface nsIDocument;
 
+// These interfaces are used by our menu code so we only have to have one
+// nsIDocumentObserver.
 
 //
 // nsIChangeObserver
@@ -74,7 +76,7 @@ interface nsIChangeObserver : nsISupports
 // to be notified when content objects change or children added/removed, etc.
 //
 // We use a central manager to redirect for performance reasons, to avoid
-// every meny/item from getting a call when any change to the document happens.
+// every menu/item from getting a call when any change to the document happens.
 //
 
 [scriptable, uuid(fb31c7e0-1dd1-11b2-8a61-f4697ef53c88)]
@@ -85,6 +87,4 @@ interface nsIChangeManager : nsISupports
   
   nsIChangeObserver Lookup ( in nsIContent aContent ) ;
   
-}; // nsPIChangeManager
-
-
+}; // nsIChangeManager

widget/src/cocoa/nsMenuBarX.mm

@@ -932,7 +932,7 @@ nsMenuBarX::ContentInserted(nsIDocument * aDocument, nsIContent * aContainer,
 // nsIChangeManager
 //
 // We don't use a |nsSupportsHashtable| because we know that the lifetime of all these items
-// is bouded by the lifetime of the menubar. No need to add any more strong refs to the
+// is bounded by the lifetime of the menubar. No need to add any more strong refs to the
 // picture because the containment hierarchy already uses strong refs.
 //
 

widget/src/cocoa/nsMenuX.mm

@@ -226,7 +226,7 @@ NS_IMETHODIMP nsMenuX::AddItem(nsISupports* aItem)
 NS_IMETHODIMP nsMenuX::AddMenuItem(nsIMenuItem * aMenuItem)
 {
   if (!aMenuItem)
-    return NS_ERROR_NULL_POINTER;
+    return NS_ERROR_INVALID_ARG;
 
   mMenuItemsArray.AppendObject(aMenuItem); // owning ref