Bug 1338086 - Remove useless else blocks in order to reduce complexity in view/ r?tnikkel
MozReview-Commit-ID: 6H794QjWZ1t
--- a/view/nsView.cpp
+++ b/view/nsView.cpp
@@ -1016,17 +1016,17 @@ nsView::WindowResized(nsIWidget* aWidget
nsIPresShell* presShell = mViewManager->GetPresShell();
if (presShell && presShell->GetDocument()) {
pm->AdjustPopupsOnWindowChange(presShell);
}
}
return true;
}
- else if (IsPopupWidget(aWidget)) {
+ if (IsPopupWidget(aWidget)) {
nsXULPopupManager* pm = nsXULPopupManager::GetInstance();
if (pm) {
pm->PopupResized(mFrame, LayoutDeviceIntSize(aWidth, aHeight));
return true;
}
}
return false;
--- a/view/nsViewManager.cpp
+++ b/view/nsViewManager.cpp
@@ -992,28 +992,28 @@ nsViewManager::SetViewVisibility(nsView
aView->SetVisibility(aVisible);
}
}
bool nsViewManager::IsViewInserted(nsView *aView)
{
if (mRootView == aView) {
return true;
- } else if (aView->GetParent() == nullptr) {
- return false;
- } else {
- nsView* view = aView->GetParent()->GetFirstChild();
- while (view != nullptr) {
- if (view == aView) {
- return true;
- }
- view = view->GetNextSibling();
- }
+ }
+ if (aView->GetParent() == nullptr) {
return false;
}
+ nsView* view = aView->GetParent()->GetFirstChild();
+ while (view != nullptr) {
+ if (view == aView) {
+ return true;
+ }
+ view = view->GetNextSibling();
+ }
+ return false;
}
void
nsViewManager::SetViewZIndex(nsView *aView, bool aAutoZIndex, int32_t aZIndex)
{
NS_ASSERTION((aView != nullptr), "no view");
// don't allow the root view's z-index to be changed. It should always be zero.