Similar to the other bugs filed (e.g., bug 74646), this is because we're mixing bit packed types. http://randomascii.wordpress.com/2010/06/06/bit-field-packing-with-visual-c/
I'm going to split this into 2 patches since there are 2 structs in RenderStyle that also need to be fixed up.
Created attachment 120045 [details] Patch
Comment on attachment 120045 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=120045&action=review > Source/WebCore/rendering/style/RenderStyle.h:169 > + bool affectedByUncommonAttributeSelectors() const { return m_affectedByUncommonAttributeSelectors; } > + void setAffectedByUncommonAttributeSelectors(bool value) { m_affectedByUncommonAttributeSelectors = value; } > + bool unique() const { return m_unique; } > + void setUnique(bool value) { m_unique = value; } > + bool affectedByEmpty() const { return m_affectedByEmpty; } > + void setAffectedByEmpty(bool value) { m_affectedByEmpty = value; } > + bool emptyState() const { return m_emptyState; } > + void setEmptyState(bool value) { m_emptyState = value; } > + bool childrenAffectedByFirstChildRules() const { return m_childrenAffectedByFirstChildRules; } > + void setChildrenAffectedByFirstChildRules(bool value) { m_childrenAffectedByFirstChildRules = value; } > + bool childrenAffectedByLastChildRules() const { return m_childrenAffectedByLastChildRules; } > + void setChildrenAffectedByLastChildRules(bool value) { m_childrenAffectedByLastChildRules = value; } > + bool childrenAffectedByDirectAdjacentRules() const { return m_childrenAffectedByDirectAdjacentRules; } > + void setChildrenAffectedByDirectAdjacentRules(bool value) { m_childrenAffectedByDirectAdjacentRules = value; } > + bool childrenAffectedByForwardPositionalRules() const { return m_childrenAffectedByForwardPositionalRules; } > + void setChildrenAffectedByForwardPositionalRules(bool value) { m_childrenAffectedByForwardPositionalRules = value; } > + bool childrenAffectedByBackwardPositionalRules() const { return m_childrenAffectedByBackwardPositionalRules; } > + void setChildrenAffectedByBackwardPositionalRules(bool value) { m_childrenAffectedByBackwardPositionalRules = value; } > + bool firstChildState() const { return m_firstChildState; } > + void setFirstChildState(bool value) { m_firstChildState = value; } > + bool lastChildState() const { return m_lastChildState; } > + void setLastChildState(bool value) { m_lastChildState = value; } > + bool explicitInheritance() const { return m_explicitInheritance; } > + void setExplicitInheritance(bool value) { m_explicitInheritance = value; } Can we generate these with a macro? Also, since all these variables are only used within getters and setters, it might make more sense to make those getters and setters friends of this class instead.
Comment on attachment 120045 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=120045&action=review >> Source/WebCore/rendering/style/RenderStyle.h:169 >> + void setExplicitInheritance(bool value) { m_explicitInheritance = value; } > > Can we generate these with a macro? Also, since all these variables are only used within getters and setters, it might make more sense to make those getters and setters friends of this class instead. Apparently we can't do this since RenderStyle hadn't be fully declared yet.
Committed r103353: <http://trac.webkit.org/changeset/103353>