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 56 matching lines @@
 #include "core/rendering/RenderMarquee.h"
 #include "core/rendering/RenderScrollbarPart.h"
 #include "core/rendering/RenderTableCaption.h"
 #include "core/rendering/RenderTableCell.h"
 #include "core/rendering/RenderTableCol.h"
 #include "core/rendering/RenderTableRow.h"
 #include "core/rendering/RenderTheme.h"
 #include "core/rendering/RenderView.h"
 #include "core/rendering/compositing/CompositedLayerMapping.h"
 #include "core/rendering/compositing/RenderLayerCompositor.h"
+#include "core/rendering/style/AppliedTextDecoration.h"
 #include "core/rendering/style/ContentData.h"
 #include "core/rendering/style/CursorList.h"
 #include "core/rendering/style/ShadowList.h"
 #include "core/rendering/svg/SVGRenderSupport.h"
 #include "platform/JSONValues.h"
 #include "platform/Partitions.h"
 #include "platform/TraceEvent.h"
 #include "platform/TracedValue.h"
 #include "platform/geometry/TransformState.h"
 #include "platform/graphics/GraphicsContext.h"
@@ skipping 2974 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)
+RenderObject::ResolvedDecoration::ResolvedDecoration()
+    : line(TextDecorationUnderline)
+    , style(TextDecorationStyleSolid)
 {
+}
+
+RenderObject::ResolvedDecoration::ResolvedDecoration(const AppliedTextDecoration& decoration, const Color& color)
+    : line(decoration.line())
+    , style(decoration.style())
+    , color(color)
+{
+}
+
+static RenderStyle* decorationStyle(RenderObject* renderObject, bool firstlineStyle, TextDecoration decoration)
+{
+    RenderStyle* styleToUse = renderObject->style(firstlineStyle);
+
+    if (styleToUse->textDecoration() == decoration)
+        return styleToUse;
+    if (!firstlineStyle)
+        return nullptr;
+
+    styleToUse = renderObject->style(false);
+    return styleToUse->textDecoration() == decoration ? styleToUse : nullptr;
+}
+
+void RenderObject::resolvedDecorations(bool firstlineStyle, const Vector<AppliedTextDecoration>& decorations, ResolvedDecorationVector& resolved)
+{
+    ASSERT(decorations.size());
+
+    bool quirksMode = document().inQuirksMode();
+
     RenderObject* curr = this;
-    RenderStyle* styleToUse = 0;
-    unsigned currDecs = TextDecorationNone;
-    Color resultColor;
-    TextDecorationStyle resultStyle;
+    Vector<AppliedTextDecoration>::const_reverse_iterator decoration = decorations.rbegin();
+
     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->isText()) {
+            RenderStyle* styleToUse = decorationStyle(curr, firstlineStyle, decoration->line());
+            if (styleToUse) {
+                resolved.append(RenderObject::ResolvedDecoration(*decoration, styleToUse->visitedDependentDecorationColor()));
+                ++decoration;
             }
         }
-        if (curr->isRubyText())
-            return;
         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 && (decoration != decorations.rend()) && (!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 && (decoration != decorations.rend())) {
+        RenderStyle* styleToUse = decorationStyle(curr, firstlineStyle, decoration->line());
+        if (styleToUse)
+            resolved.append(RenderObject::ResolvedDecoration(*decoration, styleToUse->visitedDependentDecorationColor()));
     }
 }
 
 void RenderObject::addAnnotatedRegions(Vector<AnnotatedRegionValue>& regions)
 {
     // Convert the style regions to absolute coordinates.
     if (style()->visibility() != VISIBLE || !isBox())
         return;
 
     if (style()->getDraggableRegionMode() == DraggableRegionNone)
@@ skipping 301 matching lines @@
 {
     if (object1) {
         const WebCore::RenderObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
     }
 }
 
 #endif