[table] Stretching tables when needed due to self-alignment properties

third_party/WebKit/Source/core/layout/LayoutBox.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 1299 matching lines @@
  protected:
   void willBeDestroyed() override;
 
   void insertedIntoTree() override;
   void willBeRemovedFromTree() override;
 
   void styleWillChange(StyleDifference, const ComputedStyle& newStyle) override;
   void styleDidChange(StyleDifference, const ComputedStyle* oldStyle) override;
   void updateFromStyle() override;
 
   virtual ItemPosition selfAlignmentNormalBehavior() const {

[mstensho (USE GERRIT), 2017/02/22 13:37:04]

Is this about resolving align-self:auto? This method deserves some
documentation.

[jfernandez, 2017/03/02 11:12:58]

Well, not exactly. It's a virtual method to define the 'actual
behavior' of a new keyword that CSS Alignment spec editor added
to deal with the different alignment default behavior depending
on the layout model (flex, grid, regular blocks or absolute 
positioned). 

I think the name of the function is self-explanatory, once you
realize about the existence of this new 'normal' keyword. 

-    return ItemPositionStretch;
+    return ItemPositionStart;
   }
 
   // Returns false if it could not cheaply compute the extent (e.g. fixed
   // background), in which case the returned rect may be incorrect.
   // FIXME: make this a const method once the LayoutBox reference in BoxPainter
   // is const.
   bool getBackgroundPaintedExtent(LayoutRect&) const;
   virtual bool foregroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect,
                                                  unsigned maxDepthToTest) const;
   bool computeBackgroundIsKnownToBeObscured() const override;
@@ skipping 293 matching lines @@
 
 inline bool LayoutBox::isForcedFragmentainerBreakValue(EBreak breakValue) {
   return breakValue == BreakColumn || breakValue == BreakLeft ||
          breakValue == BreakPage || breakValue == BreakRecto ||
          breakValue == BreakRight || breakValue == BreakVerso;
 }
 
 }  // namespace blink
 
 #endif  // LayoutBox_h