Clean up naming in ComputedStyle

third_party/WebKit/Source/core/style/ComputedStyle.cpp

 /*
  * Copyright (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Adobe Systems Incorporated. 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 51 matching lines @@
     unsigned m_width;
 };
 
 static_assert(sizeof(BorderValue) == sizeof(SameSizeAsBorderValue), "BorderValue should stay small");
 
 struct SameSizeAsComputedStyle : public RefCounted<SameSizeAsComputedStyle> {
     void* dataRefs[7];
     void* ownPtrs[1];
     void* dataRefSvgStyle;
 
-    struct InheritedFlags {
+    struct InheritedData {
         unsigned m_bitfields[2];
-    } inherited_flags;
+    } inherited_data;
 
-    struct NonInheritedFlags {
+    struct NonInheritedData {
         unsigned m_bitfields[3];
-    } noninherited_flags;
+    } noninherited_data;
 };
 
 static_assert(sizeof(ComputedStyle) == sizeof(SameSizeAsComputedStyle), "ComputedStyle should stay small");
 
 PassRefPtr<ComputedStyle> ComputedStyle::create()
 {
     return adoptRef(new ComputedStyle());
 }
 
 PassRefPtr<ComputedStyle> ComputedStyle::createInitialStyle()
@@ skipping 24 matching lines @@
     : m_box(initialStyle().m_box)
     , visual(initialStyle().visual)
     , m_background(initialStyle().m_background)
     , surround(initialStyle().surround)
     , rareNonInheritedData(initialStyle().rareNonInheritedData)
     , rareInheritedData(initialStyle().rareInheritedData)
     , inherited(initialStyle().inherited)
     , m_svgStyle(initialStyle().m_svgStyle)
 {
     setBitDefaults(); // Would it be faster to copy this from the default style?
-    static_assert((sizeof(InheritedFlags) <= 8), "InheritedFlags should not grow");
-    static_assert((sizeof(NonInheritedFlags) <= 12), "NonInheritedFlags should not grow");
+    static_assert((sizeof(InheritedData) <= 8), "InheritedData should not grow");
+    static_assert((sizeof(NonInheritedData) <= 12), "NonInheritedData should not grow");
 }
 
 ALWAYS_INLINE ComputedStyle::ComputedStyle(InitialStyleTag)
 {
     setBitDefaults();
 
     m_box.init();
     visual.init();
     m_background.init();
     surround.init();
@@ skipping 16 matching lines @@
 ALWAYS_INLINE ComputedStyle::ComputedStyle(const ComputedStyle& o)
     : RefCounted<ComputedStyle>()
     , m_box(o.m_box)
     , visual(o.visual)
     , m_background(o.m_background)
     , surround(o.surround)
     , rareNonInheritedData(o.rareNonInheritedData)
     , rareInheritedData(o.rareInheritedData)
     , inherited(o.inherited)
     , m_svgStyle(o.m_svgStyle)
-    , inherited_flags(o.inherited_flags)
-    , noninherited_flags(o.noninherited_flags)
+    , inherited_data(o.inherited_data)
+    , noninherited_data(o.noninherited_data)
 {
 }
 
 static StyleRecalcChange diffPseudoStyles(const ComputedStyle& oldStyle, const ComputedStyle& newStyle)
 {
     // If the pseudoStyles have changed, we want any StyleRecalcChange that is not NoChange
     // because setStyle will do the right thing with anything else.
     if (!oldStyle.hasAnyPublicPseudoStyles())
         return NoChange;
     for (PseudoId pseudoId = FirstPublicPseudoId; pseudoId < FirstInternalPseudoId; pseudoId = static_cast<PseudoId>(pseudoId + 1)) {
@@ skipping 17 matching lines @@
     if (!oldStyle && !newStyle)
         return NoChange;
 
     if (oldStyle->display() != newStyle->display()
         || oldStyle->hasPseudoStyle(PseudoIdFirstLetter) != newStyle->hasPseudoStyle(PseudoIdFirstLetter)
         || !oldStyle->contentDataEquivalent(newStyle)
         || oldStyle->hasTextCombine() != newStyle->hasTextCombine()
         || oldStyle->justifyItems() != newStyle->justifyItems()) // TODO (lajava): We must avoid this Reattach.
         return Reattach;
 
-    if (oldStyle->inheritedNotEqual(*newStyle))
+    if (!oldStyle->inheritedEqual(*newStyle))
         return Inherit;
 
     if (*oldStyle == *newStyle)
         return diffPseudoStyles(*oldStyle, *newStyle);
 
     if (oldStyle->hasExplicitlyInheritedProperties())
         return Inherit;
 
     return NoInherit;
 }
@@ skipping 58 matching lines @@
 {
     if (isAtShadowBoundary == AtShadowBoundary) {
         // Even if surrounding content is user-editable, shadow DOM should act as a single unit, and not necessarily be editable
         EUserModify currentUserModify = userModify();
         rareInheritedData = inheritParent.rareInheritedData;
         setUserModify(currentUserModify);
     } else {
         rareInheritedData = inheritParent.rareInheritedData;
     }
     inherited = inheritParent.inherited;
-    inherited_flags = inheritParent.inherited_flags;
+    inherited_data = inheritParent.inherited_data;
     if (m_svgStyle != inheritParent.m_svgStyle)
         m_svgStyle.access()->inheritFrom(inheritParent.m_svgStyle.get());
 }
 
 void ComputedStyle::copyNonInheritedFromCached(const ComputedStyle& other)
 {
     m_box = other.m_box;
     visual = other.visual;
     m_background = other.m_background;
     surround = other.surround;
     rareNonInheritedData = other.rareNonInheritedData;
 
-    // The flags are copied one-by-one because noninherited_flags contains a bunch of stuff other than real style data.
+    // The flags are copied one-by-one because noninherited_data.m_contains a bunch of stuff other than real style data.
     // See comments for each skipped flag below.
-    noninherited_flags.effectiveDisplay = other.noninherited_flags.effectiveDisplay;
-    noninherited_flags.originalDisplay = other.noninherited_flags.originalDisplay;
-    noninherited_flags.overflowX = other.noninherited_flags.overflowX;
-    noninherited_flags.overflowY = other.noninherited_flags.overflowY;
-    noninherited_flags.verticalAlign = other.noninherited_flags.verticalAlign;
-    noninherited_flags.clear = other.noninherited_flags.clear;
-    noninherited_flags.position = other.noninherited_flags.position;
-    noninherited_flags.floating = other.noninherited_flags.floating;
-    noninherited_flags.tableLayout = other.noninherited_flags.tableLayout;
-    noninherited_flags.unicodeBidi = other.noninherited_flags.unicodeBidi;
-    noninherited_flags.hasViewportUnits = other.noninherited_flags.hasViewportUnits;
-    noninherited_flags.breakBefore = other.noninherited_flags.breakBefore;
-    noninherited_flags.breakAfter = other.noninherited_flags.breakAfter;
-    noninherited_flags.breakInside = other.noninherited_flags.breakInside;
-    noninherited_flags.hasRemUnits = other.noninherited_flags.hasRemUnits;
+    noninherited_data.m_effectiveDisplay = other.noninherited_data.m_effectiveDisplay;
+    noninherited_data.m_originalDisplay = other.noninherited_data.m_originalDisplay;
+    noninherited_data.m_overflowX = other.noninherited_data.m_overflowX;
+    noninherited_data.m_overflowY = other.noninherited_data.m_overflowY;
+    noninherited_data.m_verticalAlign = other.noninherited_data.m_verticalAlign;
+    noninherited_data.m_clear = other.noninherited_data.m_clear;
+    noninherited_data.m_position = other.noninherited_data.m_position;
+    noninherited_data.m_floating = other.noninherited_data.m_floating;
+    noninherited_data.m_tableLayout = other.noninherited_data.m_tableLayout;
+    noninherited_data.m_unicodeBidi = other.noninherited_data.m_unicodeBidi;
+    noninherited_data.m_hasViewportUnits = other.noninherited_data.m_hasViewportUnits;
+    noninherited_data.m_breakBefore = other.noninherited_data.m_breakBefore;
+    noninherited_data.m_breakAfter = other.noninherited_data.m_breakAfter;
+    noninherited_data.m_breakInside = other.noninherited_data.m_breakInside;
+    noninherited_data.m_hasRemUnits = other.noninherited_data.m_hasRemUnits;
 
     // Correctly set during selector matching:
-    // noninherited_flags.styleType
-    // noninherited_flags.pseudoBits
+    // noninherited_data.m_styleType
+    // noninherited_data.m_pseudoBits
 
     // Set correctly while computing style for children:
-    // noninherited_flags.explicitInheritance
+    // noninherited_data.m_explicitInheritance
 
     // unique() styles are not cacheable.
-    ASSERT(!other.noninherited_flags.unique);
+    DCHECK(!other.noninherited_data.m_unique);
 
     // styles with non inherited properties that reference variables are not
     // cacheable.
-    ASSERT(!other.noninherited_flags.variableReference);
+    DCHECK(!other.noninherited_data.m_variableReference);
 
     // The following flags are set during matching before we decide that we get a
     // match in the MatchedPropertiesCache which in turn calls this method. The
     // reason why we don't copy these flags is that they're already correctly set
     // and that they may differ between elements which have the same set of matched
     // properties. For instance, given the rule:
     //
     // :-webkit-any(:hover, :focus) { background-color: green }"
     //
     // A hovered element, and a focused element may use the same cached matched
     // properties here, but the affectedBy flags will be set differently based on
     // the matching order of the :-webkit-any components.
     //
-    // noninherited_flags.emptyState
-    // noninherited_flags.affectedByFocus
-    // noninherited_flags.affectedByHover
-    // noninherited_flags.affectedByActive
-    // noninherited_flags.affectedByDrag
-    // noninherited_flags.isLink
+    // noninherited_data.m_emptyState
+    // noninherited_data.m_affectedByFocus
+    // noninherited_data.m_affectedByHover
+    // noninherited_data.m_affectedByActive
+    // noninherited_data.m_affectedByDrag
+    // noninherited_data.m_isLink
 
     if (m_svgStyle != other.m_svgStyle)
         m_svgStyle.access()->copyNonInheritedFromCached(other.m_svgStyle.get());
-    ASSERT(zoom() == initialZoom());
+    DCHECK_EQ(zoom(), initialZoom());
 }
 
 bool ComputedStyle::operator==(const ComputedStyle& o) const
 {

[sashab, 2016/07/01 06:59:50]

This method is identical (all comparisons are in the other methods) *except* for
the loadingCustomFonts() check in inheritedEqual(). Although this technically is
a behavior change, I doubt this actually changes anything -- two styles should
not be equal if their loadingCustomFonts() are not equal. So if anything this
fixes a bug we haven't found yet. :^)

[rune, 2016/07/01 08:45:41]

That will cause an assert in the baseComputedStyle optimization for animations
(the bots found it). We explicitly check for loadingCustomFonts() in addition to
operator== where we need to.

-    // compare everything except the pseudoStyle pointer
-    return inherited_flags == o.inherited_flags
-        && noninherited_flags == o.noninherited_flags
-        && m_box == o.m_box
-        && visual == o.visual
-        && m_background == o.m_background
-        && surround == o.surround
-        && rareNonInheritedData == o.rareNonInheritedData
-        && rareInheritedData == o.rareInheritedData
-        && inherited == o.inherited
-        && m_svgStyle == o.m_svgStyle;
+    return inheritedEqual(o)
+        && nonInheritedEqual(o);
 }
 
 bool ComputedStyle::isStyleAvailable() const
 {
     return this != StyleResolver::styleNotYetAvailable();
 }
 
 bool ComputedStyle::hasUniquePseudoStyle() const
 {
     if (!m_cachedPseudoStyles || styleType() != PseudoIdNone)
@@ skipping 48 matching lines @@
         return;
     for (size_t i = 0; i < m_cachedPseudoStyles->size(); ++i) {
         ComputedStyle* pseudoStyle = m_cachedPseudoStyles->at(i).get();
         if (pseudoStyle->styleType() == pid) {
             m_cachedPseudoStyles->remove(i);
             return;
         }
     }
 }
 
-bool ComputedStyle::inheritedNotEqual(const ComputedStyle& other) const
+bool ComputedStyle::inheritedEqual(const ComputedStyle& other) const

[sashab, 2016/07/01 06:59:50]

It's less confusing if it's an inheritedEqual() method, since
!oldStyle->inheritedNotEqual(newStyle) is used later and reads better as
oldStyle->inheritedEqual(newStyle)

 {
-    return inherited_flags != other.inherited_flags
-        || inherited != other.inherited
-        || font().loadingCustomFonts() != other.font().loadingCustomFonts()
-        || m_svgStyle->inheritedNotEqual(other.m_svgStyle.get())
-        || rareInheritedData != other.rareInheritedData;
+    return inherited_data == other.inherited_data
+        && inherited == other.inherited
+        && font().loadingCustomFonts() == other.font().loadingCustomFonts()
+        && m_svgStyle->inheritedEqual(*other.m_svgStyle)
+        && rareInheritedData == other.rareInheritedData;
+}
+
+bool ComputedStyle::nonInheritedEqual(const ComputedStyle& other) const
+{
+    // compare everything except the pseudoStyle pointer
+    return noninherited_data == other.noninherited_data
+        && m_box == other.m_box
+        && visual == other.visual
+        && m_background == other.m_background
+        && surround == other.surround
+        && rareNonInheritedData == other.rareNonInheritedData
+        && m_svgStyle->nonInheritedEqual(*other.m_svgStyle);
 }
 
 bool ComputedStyle::inheritedDataShared(const ComputedStyle& other) const
 {
     // This is a fast check that only looks if the data structures are shared.
-    return inherited_flags == other.inherited_flags
+    return inherited_data == other.inherited_data
         && inherited.get() == other.inherited.get()
         && m_svgStyle.get() == other.m_svgStyle.get()
         && rareInheritedData.get() == other.rareInheritedData.get();
 }
 
 static bool dependenceOnContentHeightHasChanged(const ComputedStyle& a, const ComputedStyle& b)
 {
     // If top or bottom become auto/non-auto then it means we either have to solve height based
     // on the content or stop doing so (http://www.w3.org/TR/CSS2/visudet.html#abs-non-replaced-height)
     // - either way requires a layout.
@@ skipping 156 matching lines @@
         return true;
 
     if (inherited.get() != other.inherited.get()) {
         if (inherited->line_height != other.inherited->line_height
             || inherited->font != other.inherited->font
             || inherited->horizontal_border_spacing != other.inherited->horizontal_border_spacing
             || inherited->vertical_border_spacing != other.inherited->vertical_border_spacing)
             return true;
     }
 
-    if (inherited_flags._box_direction != other.inherited_flags._box_direction
-        || inherited_flags.m_rtlOrdering != other.inherited_flags.m_rtlOrdering
-        || inherited_flags._text_align != other.inherited_flags._text_align
-        || inherited_flags._text_transform != other.inherited_flags._text_transform
-        || inherited_flags._direction != other.inherited_flags._direction
-        || inherited_flags._white_space != other.inherited_flags._white_space
-        || inherited_flags.m_writingMode != other.inherited_flags.m_writingMode)
+    if (inherited_data.m_boxDirection != other.inherited_data.m_boxDirection
+        || inherited_data.m_rtlOrdering != other.inherited_data.m_rtlOrdering
+        || inherited_data.m_textAlign != other.inherited_data.m_textAlign
+        || inherited_data.m_textTransform != other.inherited_data.m_textTransform
+        || inherited_data.m_direction != other.inherited_data.m_direction
+        || inherited_data.m_whiteSpace != other.inherited_data.m_whiteSpace
+        || inherited_data.m_writingMode != other.inherited_data.m_writingMode)
         return true;
 
-    if (noninherited_flags.overflowX != other.noninherited_flags.overflowX
-        || noninherited_flags.overflowY != other.noninherited_flags.overflowY
-        || noninherited_flags.clear != other.noninherited_flags.clear
-        || noninherited_flags.unicodeBidi != other.noninherited_flags.unicodeBidi
-        || noninherited_flags.floating != other.noninherited_flags.floating
-        || noninherited_flags.originalDisplay != other.noninherited_flags.originalDisplay)
+    if (noninherited_data.m_overflowX != other.noninherited_data.m_overflowX
+        || noninherited_data.m_overflowY != other.noninherited_data.m_overflowY
+        || noninherited_data.m_clear != other.noninherited_data.m_clear
+        || noninherited_data.m_unicodeBidi != other.noninherited_data.m_unicodeBidi
+        || noninherited_data.m_floating != other.noninherited_data.m_floating
+        || noninherited_data.m_originalDisplay != other.noninherited_data.m_originalDisplay)
         return true;
 
-    if (noninherited_flags.effectiveDisplay >= FIRST_TABLE_DISPLAY && noninherited_flags.effectiveDisplay <= LAST_TABLE_DISPLAY) {
-        if (inherited_flags._border_collapse != other.inherited_flags._border_collapse
-            || inherited_flags._empty_cells != other.inherited_flags._empty_cells
-            || inherited_flags._caption_side != other.inherited_flags._caption_side
-            || noninherited_flags.tableLayout != other.noninherited_flags.tableLayout)
+    if (noninherited_data.m_effectiveDisplay >= FIRST_TABLE_DISPLAY && noninherited_data.m_effectiveDisplay <= LAST_TABLE_DISPLAY) {
+        if (inherited_data.m_borderCollapse != other.inherited_data.m_borderCollapse
+            || inherited_data.m_emptyCells != other.inherited_data.m_emptyCells
+            || inherited_data.m_captionSide != other.inherited_data.m_captionSide
+            || noninherited_data.m_tableLayout != other.noninherited_data.m_tableLayout)
             return true;
 
         // In the collapsing border model, 'hidden' suppresses other borders, while 'none'
         // does not, so these style differences can be width differences.
-        if (inherited_flags._border_collapse
+        if (inherited_data.m_borderCollapse
             && ((borderTopStyle() == BorderStyleHidden && other.borderTopStyle() == BorderStyleNone)
                 || (borderTopStyle() == BorderStyleNone && other.borderTopStyle() == BorderStyleHidden)
                 || (borderBottomStyle() == BorderStyleHidden && other.borderBottomStyle() == BorderStyleNone)
                 || (borderBottomStyle() == BorderStyleNone && other.borderBottomStyle() == BorderStyleHidden)
                 || (borderLeftStyle() == BorderStyleHidden && other.borderLeftStyle() == BorderStyleNone)
                 || (borderLeftStyle() == BorderStyleNone && other.borderLeftStyle() == BorderStyleHidden)
                 || (borderRightStyle() == BorderStyleHidden && other.borderRightStyle() == BorderStyleNone)
                 || (borderRightStyle() == BorderStyleNone && other.borderRightStyle() == BorderStyleHidden)))
             return true;
-    } else if (noninherited_flags.effectiveDisplay == LIST_ITEM) {
-        if (inherited_flags._list_style_type != other.inherited_flags._list_style_type
-            || inherited_flags._list_style_position != other.inherited_flags._list_style_position)
+    } else if (noninherited_data.m_effectiveDisplay == LIST_ITEM) {
+        if (inherited_data.m_listStyleType != other.inherited_data.m_listStyleType
+            || inherited_data.m_listStylePosition != other.inherited_data.m_listStylePosition)
             return true;
     }
 
     if ((visibility() == COLLAPSE) != (other.visibility() == COLLAPSE))
         return true;
 
     if (hasPseudoStyle(PseudoIdScrollbar) != other.hasPseudoStyle(PseudoIdScrollbar))
         return true;
 
     // Movement of non-static-positioned object is special cased in ComputedStyle::visualInvalidationDiff().
@@ skipping 12 matching lines @@
             || m_box->maxHeight() != other.m_box->maxHeight())
             return true;
 
         if (m_box->verticalAlign() != other.m_box->verticalAlign())
             return true;
 
         if (m_box->boxSizing() != other.m_box->boxSizing())
             return true;
     }
 
-    if (noninherited_flags.verticalAlign != other.noninherited_flags.verticalAlign
-        || noninherited_flags.position != other.noninherited_flags.position)
+    if (noninherited_data.m_verticalAlign != other.noninherited_data.m_verticalAlign
+        || noninherited_data.m_position != other.noninherited_data.m_position)
         return true;
 
     if (surround.get() != other.surround.get()) {
         if (surround->padding != other.surround->padding)
             return true;
     }
 
     if (rareNonInheritedData.get() != other.rareNonInheritedData.get()) {
         if (rareNonInheritedData->m_alignContent != other.rareNonInheritedData->m_alignContent
             || rareNonInheritedData->m_alignItems != other.rareNonInheritedData->m_alignItems
@@ skipping 30 matching lines @@
     }
 
     return false;
 }
 
 bool ComputedStyle::diffNeedsPaintInvalidationObject(const ComputedStyle& other) const
 {
     if (!m_background->outline().visuallyEqual(other.m_background->outline()))
         return true;
 
-    if (inherited_flags._visibility != other.inherited_flags._visibility
-        || inherited_flags.m_printColorAdjust != other.inherited_flags.m_printColorAdjust
-        || inherited_flags._insideLink != other.inherited_flags._insideLink
+    if (inherited_data.m_visibility != other.inherited_data.m_visibility
+        || inherited_data.m_printColorAdjust != other.inherited_data.m_printColorAdjust
+        || inherited_data.m_insideLink != other.inherited_data.m_insideLink
         || !surround->border.visuallyEqual(other.surround->border)
         || !m_background->visuallyEqual(*other.m_background))
         return true;
 
     if (rareInheritedData.get() != other.rareInheritedData.get()) {
         if (rareInheritedData->userModify != other.rareInheritedData->userModify
             || rareInheritedData->userSelect != other.rareInheritedData->userSelect
             || rareInheritedData->m_imageRendering != other.rareInheritedData->m_imageRendering)
             return true;
     }
@@ skipping 82 matching lines @@
         if (RuntimeEnabledFeatures::cssBoxReflectFilterEnabled() && !rareNonInheritedData->reflectionDataEquivalent(*other.rareNonInheritedData.get()))
             diff.setFilterChanged();
     }
 
     if (!m_background->outline().visuallyEqual(other.m_background->outline()) || !surround->border.visualOverflowEqual(other.surround->border))
         diff.setNeedsRecomputeOverflow();
 
     if (!diff.needsPaintInvalidation()) {
         if (inherited->color != other.inherited->color
             || inherited->visitedLinkColor != other.inherited->visitedLinkColor
-            || inherited_flags.m_textUnderline != other.inherited_flags.m_textUnderline
+            || inherited_data.m_textUnderline != other.inherited_data.m_textUnderline
             || visual->textDecoration != other.visual->textDecoration) {
             diff.setTextDecorationOrColorChanged();
         } else if (rareNonInheritedData.get() != other.rareNonInheritedData.get()
             && (rareNonInheritedData->m_textDecorationStyle != other.rareNonInheritedData->m_textDecorationStyle
                 || rareNonInheritedData->m_textDecorationColor != other.rareNonInheritedData->m_textDecorationColor
                 || rareNonInheritedData->m_visitedLinkTextDecorationColor != other.rareNonInheritedData->m_visitedLinkTextDecorationColor)) {
                 diff.setTextDecorationOrColorChanged();
         } else if (rareInheritedData.get() != other.rareInheritedData.get()
             && (rareInheritedData->textFillColor() != other.rareInheritedData->textFillColor()
                 || rareInheritedData->textStrokeColor() != other.rareInheritedData->textStrokeColor()
@@ skipping 422 matching lines @@
     const Vector<AppliedTextDecoration>& applied = appliedTextDecorations();
 
     for (size_t i = 0; i < applied.size(); ++i)
         decorations |= applied[i].line();
 
     return static_cast<TextDecoration>(decorations);
 }
 
 const Vector<AppliedTextDecoration>& ComputedStyle::appliedTextDecorations() const
 {
-    if (!inherited_flags.m_textUnderline && !rareInheritedData->appliedTextDecorations) {
+    if (!inherited_data.m_textUnderline && !rareInheritedData->appliedTextDecorations) {
         DEFINE_STATIC_LOCAL(Vector<AppliedTextDecoration>, empty, ());
         return empty;
     }
-    if (inherited_flags.m_textUnderline) {
+    if (inherited_data.m_textUnderline) {
         DEFINE_STATIC_LOCAL(Vector<AppliedTextDecoration>, underline, (1, AppliedTextDecoration(TextDecorationUnderline)));
         return underline;
     }
 
     return rareInheritedData->appliedTextDecorations->vector();
 }
 
 StyleVariableData* ComputedStyle::variables() const
 {
     ASSERT(RuntimeEnabledFeatures::cssVariablesEnabled());
@@ skipping 114 matching lines @@
 
 void ComputedStyle::addAppliedTextDecoration(const AppliedTextDecoration& decoration)
 {
     RefPtr<AppliedTextDecorationList>& list = rareInheritedData.access()->appliedTextDecorations;
 
     if (!list)
         list = AppliedTextDecorationList::create();
     else if (!list->hasOneRef())
         list = list->copy();
 
-    if (inherited_flags.m_textUnderline) {
-        inherited_flags.m_textUnderline = false;
+    if (inherited_data.m_textUnderline) {
+        inherited_data.m_textUnderline = false;
         list->append(AppliedTextDecoration(TextDecorationUnderline));
     }
 
     list->append(decoration);
 }
 
 void ComputedStyle::applyTextDecorations()
 {
     if (getTextDecoration() == TextDecorationNone)
         return;
 
     TextDecorationStyle style = getTextDecorationStyle();
     StyleColor styleColor = decorationColorIncludingFallback(insideLink() == InsideVisitedLink);
 
     int decorations = getTextDecoration();
 
     if (decorations & TextDecorationUnderline) {
         // To save memory, we don't use AppliedTextDecoration objects in the
         // common case of a single simple underline.
         AppliedTextDecoration underline(TextDecorationUnderline, style, styleColor);
 
         if (!rareInheritedData->appliedTextDecorations && underline.isSimpleUnderline())
-            inherited_flags.m_textUnderline = true;
+            inherited_data.m_textUnderline = true;
         else
             addAppliedTextDecoration(underline);
     }
     if (decorations & TextDecorationOverline)
         addAppliedTextDecoration(AppliedTextDecoration(TextDecorationOverline, style, styleColor));
     if (decorations & TextDecorationLineThrough)
         addAppliedTextDecoration(AppliedTextDecoration(TextDecorationLineThrough, style, styleColor));
 }
 
 void ComputedStyle::clearAppliedTextDecorations()
 {
-    inherited_flags.m_textUnderline = false;
+    inherited_data.m_textUnderline = false;
 
     if (rareInheritedData->appliedTextDecorations)
         rareInheritedData.access()->appliedTextDecorations = nullptr;
 }
 
 void ComputedStyle::clearMultiCol()
 {
     rareNonInheritedData.access()->m_multiCol = nullptr;
     rareNonInheritedData.access()->m_multiCol.init();
 }
@@ skipping 399 matching lines @@
         if (value < 0)
             fvalue -= 0.5f;
         else
             fvalue += 0.5f;
     }
 
     return roundForImpreciseConversion<int>(fvalue / zoomFactor);
 }
 
 } // namespace blink