XFA - Remove empty pages

  - it aims to fix #13583;
  - fix the switch to breakBefore target;
  - force the layout of an unsplittable element on an empty page;
  - don't fail when there is horizontal overflow (except in lr-tb);
  - fix a typo in radial gradient first argument.

Author: calixteman

src/core/xfa/layout.js

@@ -257,25 +257,30 @@ function getTransformedBBox(node) {
  * in case of lr-tb or changing content area...).
  */
 function checkDimensions(node, space) {
+  if (node[$getTemplateRoot]()[$extra].firstUnsplittable === null) {
+    return true;
+  }
+
   if (node.w === 0 || node.h === 0) {
     return true;
   }
 
   if (space.width <= 0 || space.height <= 0) {
-    return false;
+    return node[$getTemplateRoot]()[$extra].noLayoutFailure;
   }
 
   const parent = node[$getParent]();
-  const attempt = (node[$extra] && node[$extra].attempt) || 0;
+  const attempt = (parent[$extra] && parent[$extra].attempt) || 0;
   switch (parent.layout) {
     case "lr-tb":
     case "rl-tb":
-      switch (attempt) {
-        case 0: {
-          let w, h;
-          if (node.w !== "" || node.h !== "") {
-            [, , w, h] = getTransformedBBox(node);
-          }
+      if (attempt === 0) {
+        let w, h;
+        if (node.w !== "" || node.h !== "") {
+          [, , w, h] = getTransformedBBox(node);
+        }
+
+        if (!node[$getTemplateRoot]()[$extra].noLayoutFailure) {
           if (node.h !== "" && Math.round(h - space.height) > 1) {
             return false;
           }
@@ -285,42 +290,51 @@ function checkDimensions(node, space) {
 
           return node.minW <= space.width;
         }
-        case 1: {
-          if (node.h !== "" && !node[$isSplittable]()) {
-            const [, , , h] = getTransformedBBox(node);
-            if (Math.round(h - space.height) > 1) {
-              return false;
-            }
-          }
-          return true;
+        if (node.w !== "") {
+          return Math.round(w - space.width) <= 1;
         }
-        default:
-          return true;
+        return true;
       }
-    case "table":
-    case "tb":
-      if (attempt !== 1 && node.h !== "" && !node[$isSplittable]()) {
+      if (node[$getTemplateRoot]()[$extra].noLayoutFailure) {
+        return true;
+      }
+
+      if (node.h !== "" && !node[$isSplittable]()) {
         const [, , , h] = getTransformedBBox(node);
         if (Math.round(h - space.height) > 1) {
           return false;
         }
       }
       return true;
-    case "position":
-      const [x, y, w, h] = getTransformedBBox(node);
-      const isWidthOk = node.w === "" || Math.round(w + x - space.width) <= 1;
-      const isHeightOk = node.h === "" || Math.round(h + y - space.height) <= 1;
 
-      if (isWidthOk && isHeightOk) {
+    case "table":
+    case "tb":
+      if (node[$getTemplateRoot]()[$extra].noLayoutFailure) {
         return true;
       }
 
-      const area = node[$getTemplateRoot]()[$extra].currentContentArea;
-      if (isWidthOk) {
-        return h + y > area.h;
+      // If the node has a height then check
+      // if it's fine with available height. If the node
+      // is breakable then we can return true.
+      if (node.h !== "" && !node[$isSplittable]()) {
+        const [, , , h] = getTransformedBBox(node);
+        return Math.round(h - space.height) <= 1;
+      }
+      // Else wait and see: this node will be layed out itself
+      // in the provided space and maybe a children won't fit.
+      return true;
+    case "position":
+      if (node[$getTemplateRoot]()[$extra].noLayoutFailure) {
+        return true;
+      }
+
+      const [, y, , h] = getTransformedBBox(node);
+      if (node.h === "" || Math.round(h + y - space.height) <= 1) {
+        return true;
       }
 
-      return w + x > area.w;
+      const area = node[$getTemplateRoot]()[$extra].currentContentArea;
+      return h + y > area.h;
     case "rl-row":
     case "row":
     default: