Proper support for multiple text decorations.

Source/core/rendering/RenderObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 2959 matching lines @@
     if (style->textStrokeWidth() > 0) {
         // Prefer stroke color if possible but not if it's fully transparent.
         Color textStrokeColor = object->resolveColor(style, CSSPropertyWebkitTextStrokeColor);
         if (textStrokeColor.alpha())
             return textStrokeColor;
     }
 
     return object->resolveColor(style, CSSPropertyWebkitTextFillColor);
 }
 
-void RenderObject::getTextDecorations(unsigned decorations, AppliedTextDecoration& underline, AppliedTextDecoration& overline, AppliedTextDecoration& linethrough, bool quirksMode, bool firstlineStyle)
+static bool hasSimpleUnderline(RenderStyle* style)
+{
+    return style->textDecoration() & TextDecorationUnderline
+        && style->textDecorationStyle() == TextDecorationStyleSolid;

[esprehn, 2014/03/31 17:41:24]

This should be a method on RenderStyle.

+}
+
+Color RenderObject::getUnderlineColor(bool quirksMode, bool firstlineStyle)
 {
     RenderObject* curr = this;
-    RenderStyle* styleToUse = 0;
-    unsigned currDecs = TextDecorationNone;
-    Color resultColor;
-    TextDecorationStyle resultStyle;
     do {
-        styleToUse = curr->style(firstlineStyle);
-        currDecs = styleToUse->textDecoration();
-        currDecs &= decorations;
-        resultColor = decorationColor(this, styleToUse);
-        resultStyle = styleToUse->textDecorationStyle();
-        // Parameter 'decorations' is cast as an int to enable the bitwise operations below.
-        if (currDecs) {
-            if (currDecs & TextDecorationUnderline) {
-                decorations &= ~TextDecorationUnderline;
-                underline.color = resultColor;
-                underline.style = resultStyle;
-            }
-            if (currDecs & TextDecorationOverline) {
-                decorations &= ~TextDecorationOverline;
-                overline.color = resultColor;
-                overline.style = resultStyle;
-            }
-            if (currDecs & TextDecorationLineThrough) {
-                decorations &= ~TextDecorationLineThrough;
-                linethrough.color = resultColor;
-                linethrough.style = resultStyle;
-            }
-        }
-        if (curr->isRubyText())
-            return;
+        RenderStyle* styleToUse = curr->style(firstlineStyle);
+        if (hasSimpleUnderline(styleToUse))
+            return styleToUse->visitedDependentDecorationColor();
         curr = curr->parent();
         if (curr && curr->isAnonymousBlock() && toRenderBlock(curr)->continuation())
             curr = toRenderBlock(curr)->continuation();
-    } while (curr && decorations && (!quirksMode || !curr->node() || (!isHTMLAnchorElement(*curr->node()) && !isHTMLFontElement(*curr->node()))));
+    } while (curr && (!quirksMode || !curr->node() || (!isHTMLAnchorElement(*curr->node()) && !isHTMLFontElement(*curr->node()))));
 
     // If we bailed out, use the element we bailed out at (typically a <font> or <a> element).
-    if (decorations && curr) {
-        styleToUse = curr->style(firstlineStyle);
-        resultColor = decorationColor(this, styleToUse);
-        if (decorations & TextDecorationUnderline) {
-            underline.color = resultColor;
-            underline.style = resultStyle;
-        }
-        if (decorations & TextDecorationOverline) {
-            overline.color = resultColor;
-            overline.style = resultStyle;
-        }
-        if (decorations & TextDecorationLineThrough) {
-            linethrough.color = resultColor;
-            linethrough.style = resultStyle;
-        }
-    }
+    if (curr)
+        return curr->style(firstlineStyle)->visitedDependentDecorationColor();
+
+    // If we can't find the originating element in the first-line styles, try again
+    // with the regular styles.
+    if (firstlineStyle)
+        return getUnderlineColor(quirksMode, false);
+
+    // If we reach this, it means we can't find the originating element for
+    // the applied underline. It's probably incorrect that the underline is
+    // applied.
+    ASSERT_NOT_REACHED();
+    return Color();
 }
 
 void RenderObject::addAnnotatedRegions(Vector<AnnotatedRegionValue>& regions)
 {
     // Convert the style regions to absolute coordinates.
     if (style()->visibility() != VISIBLE || !isBox())
         return;
 
     if (style()->getDraggableRegionMode() == DraggableRegionNone)
         return;
@@ skipping 284 matching lines @@
 {
     if (object1) {
         const WebCore::RenderObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
     }
 }
 
 #endif