Bug 1203417. Propagate error result from PaintTableFrame. r=seth draft
authorRobert O'Callahan <robert@ocallahan.org>
Sat, 13 Feb 2016 09:30:27 +1300
changeset 330744 3cfeb5e0e3c5387e26abba908703d21e60e4c34a
parent 330743 2f8c9be00dc9c7222c7f9131d12b0e059b0f35c0
child 514236 271264239c5a48b408f25524aadc83170ab92d30
push id10824
push userrocallahan@mozilla.com
push dateFri, 12 Feb 2016 20:32:19 +0000
reviewersseth
bugs1203417
milestone47.0a1
Bug 1203417. Propagate error result from PaintTableFrame. r=seth MozReview-Commit-ID: 4rRkHXUl9iE
layout/tables/nsTablePainter.cpp
--- a/layout/tables/nsTablePainter.cpp
+++ b/layout/tables/nsTablePainter.cpp
@@ -278,18 +278,19 @@ TableBackgroundPainter::PaintTable(nsTab
     if (aPaintTableBackground) {
       PaintTableFrame(aTableFrame, nullptr, nullptr, nsMargin(0,0,0,0));
     }
     /* No cells; nothing else to paint */
     return result;
   }
 
   if (aPaintTableBackground) {
-    PaintTableFrame(aTableFrame, rowGroups[0], rowGroups[rowGroups.Length() - 1],
-                    aDeflate);
+    result &=
+      PaintTableFrame(aTableFrame, rowGroups[0], rowGroups[rowGroups.Length() - 1],
+                      aDeflate);
   }
 
   /*Set up column background/border data*/
   if (mNumCols > 0) {
     nsFrameList& colGroupList = aTableFrame->GetColGroups();
     NS_ASSERTION(colGroupList.FirstChild(), "table should have at least one colgroup");
 
     // Collect all col group frames first so that we know how many there are.