Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1199)

Unified Diff: third_party/WebKit/Source/core/layout/CollapsedBorderValue.h

Issue 2896783002: Remove dependency on order of enum EBorderStyle (Closed)
Patch Set: Merge branch and make it dependent on https://codereview.chromium.org/2895933004 Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/layout/CollapsedBorderValue.h
diff --git a/third_party/WebKit/Source/core/layout/CollapsedBorderValue.h b/third_party/WebKit/Source/core/layout/CollapsedBorderValue.h
index 4cd2318cfb435519957afc02737e3942887f6e78..74c55752ee67dddc67587f1710082d3fe147ef3f 100644
--- a/third_party/WebKit/Source/core/layout/CollapsedBorderValue.h
+++ b/third_party/WebKit/Source/core/layout/CollapsedBorderValue.h
@@ -25,7 +25,7 @@
#ifndef CollapsedBorderValue_h
#define CollapsedBorderValue_h
-#include "core/style/BorderValue.h"
+#include "core/style/ComputedStyle.h"
#include "platform/wtf/Allocator.h"
namespace blink {
@@ -55,7 +55,9 @@ class CollapsedBorderValue {
const Color& color,
EBorderPrecedence precedence)
: color_(color),
- width_(border.Style() > EBorderStyle::kHidden ? border.Width() : 0),
+ width_(ComputedStyle::BorderStyleIsVisible(border.Style())
+ ? border.Width()
+ : 0),
style_(static_cast<unsigned>(border.Style())),
precedence_(precedence) {
DCHECK(precedence != kBorderPrecedenceOff);
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698