remove staticLayoutContribution

this is for spreadsheets in CellEngine when you double-click a column to
fit it to text, but lineboxes are going away soon since they aren't
needed in painting. CellEngine can pretty easily just find the max of
items[i].x + items[i].measure() instead. I don't think this API made
much sense generally (only in relation to this one use case) and it has
some TODOs that would be kinda hard.

Author: chearon

README.md

@@ -709,18 +709,6 @@ environment.wasmLocator = function () {
 };
 ```
 
-## Other
-
-### `staticLayoutContribution`
-
-```ts
-function staticLayoutContribution(box: BlockContainer): number;
-```
-
-Returns the inline size in CSS pixels taken up by the layout, not including empty space after lines or the effect of any `width` properties. `layout` must be called before this.
-
-The intended usage is this: after laying out text into a desired size, use `staticLayoutContribution` to get the size without any remaining empty space at the end of the lines, then `layout` again into that size to get a tightly fitting layout.
-
 # HarfBuzz
 
 Glyph layout is performed by [HarfBuzz](https://github.com/harfbuzz/harfbuzz) compiled to WebAssembly. This allows for a level of correctness that isn't possible by using the `measureText` API to position spans of text. If you color the "V" in the text "AV" differently in Google Sheets, you will notice kerning is lost, and the letters appear further apart than they should be. That's because two `measureText` and `fillText` calls were made on the letters, so contextual glyph advances were lost. Dropflow uses HarfBuzz on more coarse shaping boundaries (not when color is changed) so that the font is more correctly supported. 

src/api.ts

@@ -212,53 +212,6 @@ export function t(text: string): TextNode {
   return new TextNode(id(), text);
 }
 
-export function staticLayoutContribution(layout: Layout, box: BlockContainer): number {
-  let intrinsicSize = 0;
-
-  const containingBlock = box.getContainingBlock();
-  const definiteSize = box.getDefiniteOuterInlineSize(containingBlock);
-  if (definiteSize !== undefined) {
-    const marginLineLeft = box.style.getMarginLineLeft(containingBlock);
-    const marginLineRight = box.style.getMarginLineRight(containingBlock);
-    return definiteSize + (marginLineLeft === 'auto' ? 0 : marginLineLeft)
-      + (marginLineRight === 'auto' ? 0 : marginLineRight)
-  }
-
-  if (box.isBlockContainerOfInlines()) {
-    for (const line of box.lineboxes) {
-      intrinsicSize = Math.max(intrinsicSize, line.width);
-    }
-    // TODO: floats
-  } else {
-    for (let i = box.treeStart + 1; i <= box.treeFinal; i++) {
-      const child = layout.tree[i];
-      if (child.isBlockContainer()) {
-        intrinsicSize = Math.max(intrinsicSize, staticLayoutContribution(layout, child));
-      } else if (child.isBox()) {
-        // TODO:
-        intrinsicSize = Math.max(intrinsicSize, child.getBorderArea().inlineSize);
-      }
-      if (child.isBox()) i = child.treeFinal;
-    }
-  }
-
-  const marginLineLeft = box.style.getMarginLineLeft(containingBlock);
-  const marginLineRight = box.style.getMarginLineRight(containingBlock);
-  const borderLineLeftWidth = box.style.getBorderLineLeftWidth(containingBlock);
-  const paddingLineLeft = box.style.getPaddingLineLeft(containingBlock);
-  const paddingLineRight = box.style.getPaddingLineRight(containingBlock);
-  const borderLineRightWidth = box.style.getBorderLineRightWidth(containingBlock);
-
-  intrinsicSize += (marginLineLeft === 'auto' ? 0 : marginLineLeft)
-    + borderLineLeftWidth
-    + paddingLineLeft
-    + paddingLineRight
-    + borderLineRightWidth
-    + (marginLineRight === 'auto' ? 0 : marginLineRight);
-
-  return intrinsicSize;
-}
-
 type LoadableResource = FontFace | Image;
 
 export interface LoadWalkerContext {