Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2013 Google Inc. All rights reserved. | 2 * Copyright (C) 2013 Google Inc. All rights reserved. |
| 3 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) | 3 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) |
| 4 * (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com) | 4 * (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com) |
| 5 * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com) | 5 * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com) |
| 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. | 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. |
| 7 * All rights reserved. | 7 * All rights reserved. |
| 8 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> | 8 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> |
| 9 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org> | 9 * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org> |
| 10 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. | 10 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. |
| (...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 85 | 85 |
| 86 namespace { | 86 namespace { |
| 87 | 87 |
| 88 static inline bool isValidVisitedLinkProperty(CSSPropertyID id) { | 88 static inline bool isValidVisitedLinkProperty(CSSPropertyID id) { |
| 89 switch (id) { | 89 switch (id) { |
| 90 case CSSPropertyBackgroundColor: | 90 case CSSPropertyBackgroundColor: |
| 91 case CSSPropertyBorderLeftColor: | 91 case CSSPropertyBorderLeftColor: |
| 92 case CSSPropertyBorderRightColor: | 92 case CSSPropertyBorderRightColor: |
| 93 case CSSPropertyBorderTopColor: | 93 case CSSPropertyBorderTopColor: |
| 94 case CSSPropertyBorderBottomColor: | 94 case CSSPropertyBorderBottomColor: |
| 95 case CSSPropertyCaretColor: | |
|
Timothy Loh
2016/11/22 02:17:35
Do we need to do this? What are the use cases for
Manuel Rego
2016/11/22 09:36:01
We actually have a test for this specific behavior
| |
| 95 case CSSPropertyColor: | 96 case CSSPropertyColor: |
| 96 case CSSPropertyFill: | 97 case CSSPropertyFill: |
| 97 case CSSPropertyOutlineColor: | 98 case CSSPropertyOutlineColor: |
| 98 case CSSPropertyStroke: | 99 case CSSPropertyStroke: |
| 99 case CSSPropertyTextDecorationColor: | 100 case CSSPropertyTextDecorationColor: |
| 100 case CSSPropertyColumnRuleColor: | 101 case CSSPropertyColumnRuleColor: |
| 101 case CSSPropertyWebkitTextEmphasisColor: | 102 case CSSPropertyWebkitTextEmphasisColor: |
| 102 case CSSPropertyWebkitTextFillColor: | 103 case CSSPropertyWebkitTextFillColor: |
| 103 case CSSPropertyWebkitTextStrokeColor: | 104 case CSSPropertyWebkitTextStrokeColor: |
| 104 return true; | 105 return true; |
| (...skipping 930 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1035 } | 1036 } |
| 1036 } | 1037 } |
| 1037 | 1038 |
| 1038 void StyleBuilderFunctions::applyInheritCSSPropertyPosition( | 1039 void StyleBuilderFunctions::applyInheritCSSPropertyPosition( |
| 1039 StyleResolverState& state) { | 1040 StyleResolverState& state) { |
| 1040 if (!state.parentNode()->isDocumentNode()) | 1041 if (!state.parentNode()->isDocumentNode()) |
| 1041 state.style()->setPosition(state.parentStyle()->position()); | 1042 state.style()->setPosition(state.parentStyle()->position()); |
| 1042 } | 1043 } |
| 1043 | 1044 |
| 1044 } // namespace blink | 1045 } // namespace blink |
| OLD | NEW |