SVG: Promote d to a property

third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
@@ skipping 57 matching lines @@
 #include "core/css/resolver/TransformBuilder.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/style/CounterContent.h"
 #include "core/style/ComputedStyle.h"
 #include "core/style/ComputedStyleConstants.h"
 #include "core/style/PathStyleMotionPath.h"
 #include "core/style/QuotesData.h"
 #include "core/style/SVGComputedStyle.h"
 #include "core/style/StyleGeneratedImage.h"
+#include "core/svg/SVGPathUtilities.h"
 #include "platform/fonts/FontDescription.h"
 #include "wtf/MathExtras.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 namespace {
 
 static inline bool isValidVisitedLinkProperty(CSSPropertyID id)
@@ skipping 773 matching lines @@
         svgStyle.setBaselineShift(BS_SUB);
         return;
     case CSSValueSuper:
         svgStyle.setBaselineShift(BS_SUPER);
         return;
     default:
         ASSERT_NOT_REACHED();
     }
 }
 
+void StyleBuilderFunctions::applyInitialCSSPropertyD(StyleResolverState& state)
+{
+    state.style()->accessSVGStyle().setD(SVGComputedStyle::initialD());
+}
+
+void StyleBuilderFunctions::applyInheritCSSPropertyD(StyleResolverState& state)
+{
+    state.style()->accessSVGStyle().setD(state.parentStyle()->svgStyle().d());
+}
+
+void StyleBuilderFunctions::applyValueCSSPropertyD(StyleResolverState& state, CSSValue* value)

[fs, 2015/11/26 12:51:39]

Feels like it should be possible to just have a converter for paths and use that
(should apply to motion-path too I think, but maybe the 'none' makes that
troublesome?) 

+{
+    AtomicString pathString = StyleBuilderConverter::convertString<CSSValueInvalid>(state, *value);
+
+    OwnPtr<SVGPathByteStream> pathByteStream = SVGPathByteStream::create();
+    buildByteStreamFromString(pathString, *pathByteStream);
+    state.style()->accessSVGStyle().setD(pathByteStream.get());
+}
+
 } // namespace blink