Move line/continuation specific parts of willBeDestroyed() into LayoutBlockFlow.

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

Index: third_party/WebKit/Source/core/layout/LayoutBlock.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlock.h b/third_party/WebKit/Source/core/layout/LayoutBlock.h
index 1d19fc3b77581a281cfda8f92add6ebc04dcd66d..83c9f4ff40e6d6e102e8bbab516b82ef072cb8c1 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlock.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBlock.h
@@ -124,8 +124,6 @@ public:
     const LayoutObjectChildList* children() const { return &m_children; }
     LayoutObjectChildList* children() { return &m_children; }
 
-    bool beingDestroyed() const { return m_beingDestroyed; }

[szager1, 2016/05/16 16:56:40]

Sorry to jump in here, but I don't like moving this method to LayoutBlockFlow
while the bitfield is still defined on LayoutBlock.  Can we leave it here?

[mstensho (USE GERRIT), 2016/05/16 17:57:27]

I moved it because the flag is now only set by LayoutBlockFlow, so it would
always be false for other types of LayoutBlock - which kind of seemed misleading
and unuseful to me.

Also note that there's nothing unusual about having bitfields in LayoutBlock
that are only used in subclasses, though. Establishing a bit field typically
requires the size of an int, so keeping them all in LayoutBlock makes sense,
when there are so few of them.

See m_hasMarkupTruncation, m_isSelfCollapsing,
m_descendantsWithFloatsMarkedForLayout...

-
     // These two functions are overridden for inline-block.
     LayoutUnit lineHeight(bool firstLine, LineDirectionMode, LinePositionMode = PositionOnContainingLine) const final;
     int baselinePosition(FontBaseline, bool firstLine, LineDirectionMode, LinePositionMode = PositionOnContainingLine) const override;