Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. r=xidorn
authorJeremy Chen <jeremychen@mozilla.com>
Wed, 07 Sep 2016 10:20:16 +0800
changeset 313040 907028433e467a3eb04cd69db622302238edc598
parent 313039 787d895935668086fb3cb693052abb48f7040759
child 313041 cd5cf15a6fa9012624514996352e4f60f9b6be9e
push id20479
push userkwierso@gmail.com
push dateThu, 08 Sep 2016 01:08:46 +0000
treeherderfx-team@fb7c6b034329 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersxidorn
bugs1297306, 956447
milestone51.0a1
Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. r=xidorn This should've been covered in Bug 956447. The declaration is already behind DEBUG_FRAME_DUMP in nsLineBox.h. Just move the implementation to agreee with that. MozReview-Commit-ID: 9N0WxKkajF1
layout/generic/nsLineBox.cpp
--- a/layout/generic/nsLineBox.cpp
+++ b/layout/generic/nsLineBox.cpp
@@ -277,19 +277,17 @@ nsLineBox::List(FILE* out, const char* a
   }
 
   if (HasFloats()) {
     fprintf_stderr(out, "%s> floats <\n", aPrefix);
     ListFloats(out, pfx.get(), mInlineData->mFloats);
   }
   fprintf_stderr(out, "%s>\n", aPrefix);
 }
-#endif
 
-#ifdef DEBUG
 nsIFrame*
 nsLineBox::LastChild() const
 {
   nsIFrame* frame = mFirstChild;
   int32_t n = GetChildCount() - 1;
   while (--n >= 0) {
     frame = frame->GetNextSibling();
   }