Bug 1330225. Remove unused function nsViewManager::InsertChild. r=mats

tnikkel · tnikkel · commit 1b21b07765a4 · 2017-01-12T00:59:11.000-06:00
diff --git a/view/nsViewManager.cpp b/view/nsViewManager.cpp
@@ -936,15 +936,6 @@ nsViewManager::InsertChild(nsView *aParent, nsView *aChild, nsView *aSibling,
     }
 }
 
-void
-nsViewManager::InsertChild(nsView *aParent, nsView *aChild, int32_t aZIndex)
-{
-  // no-one really calls this with anything other than aZIndex == 0 on a fresh view
-  // XXX this method should simply be eliminated and its callers redirected to the real method
-  SetViewZIndex(aChild, false, aZIndex);
-  InsertChild(aParent, aChild, nullptr, true);
-}
-
 void
 nsViewManager::RemoveChild(nsView *aChild)
 {
diff --git a/view/nsViewManager.h b/view/nsViewManager.h
@@ -148,8 +148,6 @@ class nsViewManager final
   void InsertChild(nsView *aParent, nsView *aChild, nsView *aSibling,
                    bool aAfter);
 
-  void InsertChild(nsView *aParent, nsView *aChild, int32_t aZIndex);
-
   /**
    * Remove a specific child view from its parent. This will NOT remove its placeholder
    * if there is one.

Original file line number	Diff line number	Diff line change
`@@ -936,15 +936,6 @@ nsViewManager::InsertChild(nsView aParent, nsView aChild, nsView *aSibling,`
`936`	`936`	`}`
`937`	`937`	`}`
`938`	`938`
`939`		`-void`
`940`		`-nsViewManager::InsertChild(nsView aParent, nsView aChild, int32_t aZIndex)`
`941`		`-{`
`942`		`- // no-one really calls this with anything other than aZIndex == 0 on a fresh view`
`943`		`- // XXX this method should simply be eliminated and its callers redirected to the real method`
`944`		`- SetViewZIndex(aChild, false, aZIndex);`
`945`		`- InsertChild(aParent, aChild, nullptr, true);`
`946`		`-}`
`947`		`-`
`948`	`939`	`void`
`949`	`940`	`nsViewManager::RemoveChild(nsView *aChild)`
`950`	`941`	`{`