Implements CSSPropertyAPI for the d property

third_party/WebKit/Source/core/css/properties/CSSPropertyAPID.cpp

+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "core/css/properties/CSSPropertyAPID.h"
+
+#include "core/css/properties/CSSPropertyOffsetPathUtils.h"
+

[Bugs Nash, 2017/05/11 23:14:53]

I would forward declare the classes CSSValue, CSSParserTokenRange,
CSSParserContext, and CSSPropertyID (even though it currently compiles, future
changes to other files could cause a compile error here).

[Eric Willigers, 2017/05/12 00:16:13]

I respectfully disagree.

Is it OK that CSSPropertyAPID.h is the first include in CSSPropertyAPID.cpp? I
assume yes.

Is CSSPropertyAPID::parseSingleValue declared in CSSPropertyAPID.h? Yes.

Thus CSSValue, CSSParserTokenRange, CSSParserContext must be forward
declared or defined, and CSSPropertyID must be defined, by the time
class CSSPropertyAPID is defined in CSSPropertyAPID.h.

Forward declaring again in CSSPropertyAPID.cpp can only ever be
redundant while these types appear in the method signature
const CSSValue* CSSPropertyAPID::parseSingleValue(
CSSParserTokenRange& range, const CSSParserContext&, CSSPropertyID)

+namespace blink {
+
+const CSSValue* CSSPropertyAPID::parseSingleValue(CSSParserTokenRange& range,
+                                                  const CSSParserContext&,
+                                                  CSSPropertyID) {
+  return CSSPropertyOffsetPathUtils::ConsumePathOrNone(range);
+}
+
+}  // namespace blink