Bug 1587905 - Remove useless variable 'subdocFrame' declaration & assignation r=emilio
authorSylvestre Ledru <sledru@mozilla.com>
Fri, 11 Oct 2019 07:34:05 +0000
changeset 497265 f0ce1a1f84cba2d5313fa46adbc8e04edec85835
parent 497264 f8479467f708eae59cd544baecf59705e8971077
child 497266 ace16b7e7aa6fe5233bc9a0ddbf9923a04cfadb7
push id97763
push usersledru@mozilla.com
push dateFri, 11 Oct 2019 07:35:35 +0000
treeherderautoland@9537a9c43cb4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1587905
milestone71.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1587905 - Remove useless variable 'subdocFrame' declaration & assignation r=emilio Differential Revision: https://phabricator.services.mozilla.com/D48889
layout/generic/nsFrameSetFrame.cpp
--- a/layout/generic/nsFrameSetFrame.cpp
+++ b/layout/generic/nsFrameSetFrame.cpp
@@ -929,21 +929,20 @@ void nsHTMLFramesetFrame::Reflow(nsPresC
     }
 
     ReflowPlaceChild(child, aPresContext, aReflowInput, offset, size,
                      &cellIndex);
 
     if (firstTime) {
       int32_t childVis;
       nsHTMLFramesetFrame* framesetFrame = do_QueryFrame(child);
-      nsSubDocumentFrame* subdocFrame;
       if (framesetFrame) {
         childVis = framesetFrame->mEdgeVisibility;
         mChildBorderColors[childX] = framesetFrame->mEdgeColors;
-      } else if ((subdocFrame = do_QueryFrame(child))) {
+      } else if (child->IsSubDocumentFrame()) {
         if (eFrameborder_Yes == mChildFrameborder[childX]) {
           childVis = ALL_VIS;
         } else if (eFrameborder_No == mChildFrameborder[childX]) {
           childVis = NONE_VIS;
         } else {  // notset
           childVis = (eFrameborder_No == frameborder) ? NONE_VIS : ALL_VIS;
         }
       } else {  // blank