From 7a51cf15978af13260347cb2573ae608e7aafc7c Mon Sep 17 00:00:00 2001
From: "dbaron%dbaron.org" <dbaron%dbaron.org>
Date: Mon, 2 Jun 2003 02:39:26 +0000
Subject: [PATCH] Replace nsSliderFrame::GetContentOf with static
 GetContentOfBox to fix hiding warning and avoid unnecessary |this| argument. 
 b=42634  Patch from Marc Loiselle <mloiselle@yahoo.com>.  r=dbaron  sr=bryner

---
 layout/xul/base/src/nsSliderFrame.cpp | 30 ++++++++++++++-------------
 layout/xul/base/src/nsSliderFrame.h   |  1 -
 2 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/layout/xul/base/src/nsSliderFrame.cpp b/layout/xul/base/src/nsSliderFrame.cpp
index a0f1871aa1e..08d29e4e8e4 100644
--- a/layout/xul/base/src/nsSliderFrame.cpp
+++ b/layout/xul/base/src/nsSliderFrame.cpp
@@ -82,6 +82,16 @@
 // Turn this on if you want to debug slider frames.
 #undef DEBUG_SLIDER
 
+static already_AddRefed<nsIContent>
+GetContentOfBox(nsIBox *aBox)
+{
+  nsIFrame *frame;
+  nsIContent *content;
+  aBox->GetFrame(&frame);
+  frame->GetContent(&content);
+  return content;
+}
+
 nsresult
 NS_NewSliderFrame ( nsIPresShell* aPresShell, nsIFrame** aNewFrame)
 {
@@ -244,7 +254,7 @@ nsSliderFrame::AttributeChanged(nsIPresContext* aPresContext,
 
       nsIBox* scrollbarBox = GetScrollbar();
       nsCOMPtr<nsIContent> scrollbar;
-      GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+      scrollbar = GetContentOfBox(scrollbarBox);
       PRInt32 current = GetCurrentPosition(scrollbar);
       PRInt32 max = GetMaxPosition(scrollbar);
       if (current < 0 || current > max)
@@ -345,7 +355,7 @@ nsSliderFrame::DoLayout(nsBoxLayoutState& aState)
   // get the scrollbar
   nsIBox* scrollbarBox = GetScrollbar();
   nsCOMPtr<nsIContent> scrollbar;
-  GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+  scrollbar = GetContentOfBox(scrollbarBox);
   PRBool isHorizontal = IsHorizontal();
 
   // get the thumb's pref size
@@ -454,7 +464,7 @@ nsSliderFrame::HandleEvent(nsIPresContext* aPresContext,
 {
   nsIBox* scrollbarBox = GetScrollbar();
   nsCOMPtr<nsIContent> scrollbar;
-  GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+  scrollbar = GetContentOfBox(scrollbarBox);
   PRBool isHorizontal = IsHorizontal();
 
   if (isDraggingThumb(aPresContext))
@@ -739,14 +749,6 @@ nsSliderFrame::GetScrollbar()
    return ibox;
 }
 
-void
-nsSliderFrame::GetContentOf(nsIBox* aBox, nsIContent** aContent)
-{
-   nsIFrame* frame = nsnull;
-   aBox->GetFrame(&frame);
-   frame->GetContent(aContent);
-}
-
 void
 nsSliderFrame::PageUpDown(nsIFrame* aThumbFrame, nscoord change)
 {
@@ -755,7 +757,7 @@ nsSliderFrame::PageUpDown(nsIFrame* aThumbFrame, nscoord change)
   // get the values from our own node.
   nsIBox* scrollbarBox = GetScrollbar();
   nsCOMPtr<nsIContent> scrollbar;
-  GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+  scrollbar = GetContentOfBox(scrollbarBox);
 
   if (mScrollbarListener)
     mScrollbarListener->PagedUpDown(); // Let the listener decide our increment.
@@ -771,7 +773,7 @@ nsSliderFrame::CurrentPositionChanged(nsIPresContext* aPresContext)
 {
   nsIBox* scrollbarBox = GetScrollbar();
   nsCOMPtr<nsIContent> scrollbar;
-  GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+  scrollbar = GetContentOfBox(scrollbarBox);
 
   PRBool isHorizontal = IsHorizontal();
 
@@ -1024,7 +1026,7 @@ nsSliderFrame::MouseDown(nsIDOMEvent* aMouseEvent)
 
     nsIBox* scrollbarBox = GetScrollbar();
     nsCOMPtr<nsIContent> scrollbar;
-    GetContentOf(scrollbarBox, getter_AddRefs(scrollbar));
+    scrollbar = GetContentOfBox(scrollbarBox);
 
     // set it
     SetCurrentPosition(scrollbar, thumbFrame, pospx, PR_FALSE);
diff --git a/layout/xul/base/src/nsSliderFrame.h b/layout/xul/base/src/nsSliderFrame.h
index 13a2000912f..bd079d535ca 100644
--- a/layout/xul/base/src/nsSliderFrame.h
+++ b/layout/xul/base/src/nsSliderFrame.h
@@ -232,7 +232,6 @@ public:
 private:
 
   nsIBox* GetScrollbar();
-  void GetContentOf(nsIBox* aBox, nsIContent** aContent);
 
   void PageUpDown(nsIFrame* aThumbFrame, nscoord change);
   void SetCurrentPosition(nsIContent* scrollbar, nsIFrame* aThumbFrame, nscoord pos, PRBool aIsSmooth);