Tweaks to the interfaces to make them internally consistent.

widget/public/nsIDMItem.h

@@ -34,17 +34,14 @@ class nsIDMItem : public nsISupports
 {
 public:
 	// Inspectors
-	NS_IMETHOD GetIconImage(nsIImage** pImage, nsIImageGroup* pGroup) = 0;
+	NS_IMETHOD GetIconImage(nsIImage*& pImage, nsIImageGroup* pGroup) = 0;
 	NS_IMETHOD IsExpanded(PRBool& answer) = 0;
 	
 	// Setters
 
 	// Methods to query the data model for a specific item displayed within the widget.
-	NS_IMETHOD GetStringPropertyValue(PRBool& answer, const nsString& itemProperty,
-								      nsString& value) = 0;
-	NS_IMETHOD GetIntPropertyValue(PRBool& answer, const nsString& itemProperty,
-								      int& value) = 0;
-	
+	NS_IMETHOD GetStringPropertyValue(nsString& value, const nsString& itemProperty) = 0;
+	NS_IMETHOD GetIntPropertyValue(int& value, const nsString& itemProperty) = 0;
 };
 
 #endif /* nsIDMItem_h___ */

widget/public/nsIDataModel.h

@@ -38,10 +38,8 @@ public:
 	NS_IMETHOD SetDMWidget(nsIDMWidget* pWidget) = 0;
 
 	// Methods to query the data model for property values for an entire widget.
-	NS_IMETHOD GetStringPropertyValue(PRBool& answer,
-					       			  const nsString& property, const nsString& value) = 0;
-	NS_IMETHOD GetIntPropertyValue(PRBool& answer,
-								   const nsString& property, int& value) = 0;
+	NS_IMETHOD GetStringPropertyValue(nsString& value, const nsString& property) = 0;
+	NS_IMETHOD GetIntPropertyValue(int& value, const nsString& property) = 0;
 };
 
 #endif /* nsIDataModel_h___ */

widget/public/nsITreeColumn.h

@@ -33,7 +33,7 @@ public:
 	// Inspectors
 	NS_IMETHOD GetPixelWidth(int& width) = 0;
 	NS_IMETHOD GetDesiredPercentage(double& percentage) = 0;
-	NS_IMETHOD IsSortColumn(PRBool& answer) = 0;
+	NS_IMETHOD GetSortState(int& answer) = 0;
 	NS_IMETHOD GetColumnName(nsString& name) = 0;
 
 	// Setters

widget/public/nsITreeDataModel.h

@@ -36,13 +36,12 @@ class nsITreeDataModel : public nsIDataModel
 public:
 	// Column APIs
 	NS_IMETHOD GetVisibleColumnCount(int& count) = 0;
-	NS_IMETHOD GetNthColumn(nsITreeColumn** pColumn, int n) = 0;
+	NS_IMETHOD GetNthColumn(nsITreeColumn*& pColumn, int n) = 0;
 	
 	// TreeItem APIs
 	NS_IMETHOD GetFirstVisibleItemIndex(int& index) = 0;
-	NS_IMETHOD GetNthTreeItem(nsITreeItem** pItem, int n) = 0;
-	NS_IMETHOD GetItemTextForColumn(nsITreeItem* pItem, nsITreeColumn* pColumn, 
-									nsString& nodeText) = 0;
+	NS_IMETHOD GetNthTreeItem(nsITreeItem*& pItem, int n) = 0;
+	NS_IMETHOD GetItemTextForColumn(nsString& nodeText, nsITreeItem* pItem, nsITreeColumn* pColumn) = 0;
 };
 
 #endif /* nsITreeDataModel_h___ */

widget/public/nsITreeItem.h

@@ -34,7 +34,7 @@ class nsITreeItem : public nsIDMItem
 {
 public:
 	// Inspectors
-	NS_IMETHOD GetTriggerImage(nsIImage** pImage, nsIImageGroup* pGroup) = 0;
+	NS_IMETHOD GetTriggerImage(nsIImage*& pImage, nsIImageGroup* pGroup) = 0;
 	NS_IMETHOD GetIndentationLevel(int& indentation) = 0;
 	
 	// Setters