Commit 4fca35a0 authored by Darren Shen's avatar Darren Shen Committed by Commit Bot

Remove unused auto value in EBorderStyle.

Since OutlineValue has been deleted, whether an outline is auto or not
is now stored in a separate field. This means we no longer need
EBorderStyle to store it. This patch removes the unused auto value
in EBorderStyle.

Change-Id: I3d698a65f6b0a8a1e2b4bc7cb2e1279e410a7461
Reviewed-on: https://chromium-review.googlesource.com/560807
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: default avatarnainar <nainar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485541}
parent 2db51a01
......@@ -37,9 +37,7 @@ class BorderStyle {
friend class ComputedStyle;
public:
BorderStyle()
: style_(static_cast<unsigned>(EBorderStyle::kNone)),
is_auto_(kOutlineIsAutoOff) {}
BorderStyle() : style_(static_cast<unsigned>(EBorderStyle::kNone)) {}
bool NonZero() const {
return (style_ != static_cast<unsigned>(EBorderStyle::kNone));
......@@ -64,14 +62,8 @@ class BorderStyle {
EBorderStyle Style() const { return static_cast<EBorderStyle>(style_); }
void SetStyle(EBorderStyle style) { style_ = static_cast<unsigned>(style); }
OutlineIsAuto IsAuto() const { return static_cast<OutlineIsAuto>(is_auto_); }
void SetIsAuto(OutlineIsAuto is_auto) { is_auto_ = is_auto; }
protected:
unsigned style_ : 4; // EBorderStyle
// This is only used by OutlineValue but moved here to keep the bits packed.
unsigned is_auto_ : 1; // OutlineIsAuto
};
} // namespace blink
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment