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

Side by Side Diff: third_party/WebKit/Source/core/style/CollapsedBorderValue.h

Issue 1574933002: Changed type of border-width longhands from unsigned to float. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made transitions use fractional border widths, added assert that border widths are positive, and r… Created 4 years, 11 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Antti Koivisto (koivisto@kde.org) 3 * (C) 2000 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2003, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
6 * Copyright (C) 2006 Graham Dennis (graham.dennis@gmail.com) 6 * Copyright (C) 2006 Graham Dennis (graham.dennis@gmail.com)
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 } 71 }
72 72
73 bool shouldPaint(const CollapsedBorderValue& tableCurrentBorderValue) const 73 bool shouldPaint(const CollapsedBorderValue& tableCurrentBorderValue) const
74 { 74 {
75 return style() > BHIDDEN && !isTransparent() && exists() && isSameIgnori ngColor(tableCurrentBorderValue); 75 return style() > BHIDDEN && !isTransparent() && exists() && isSameIgnori ngColor(tableCurrentBorderValue);
76 } 76 }
77 77
78 private: 78 private:
79 Color m_color; 79 Color m_color;
80 unsigned m_colorIsCurrentColor : 1; 80 unsigned m_colorIsCurrentColor : 1;
81 // TODO(bugsnash): change m_width to float
81 unsigned m_width : 23; 82 unsigned m_width : 23;
82 unsigned m_style : 4; // EBorderStyle 83 unsigned m_style : 4; // EBorderStyle
83 unsigned m_precedence : 3; // EBorderPrecedence 84 unsigned m_precedence : 3; // EBorderPrecedence
84 unsigned m_transparent : 1; 85 unsigned m_transparent : 1;
85 }; 86 };
86 87
87 } // namespace blink 88 } // namespace blink
88 89
89 #endif // CollapsedBorderValue_h 90 #endif // CollapsedBorderValue_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698