Use an SVGElementProxy in ReferenceClipPathOperation

third_party/WebKit/Source/core/style/ClipPathOperation.cpp

Index: third_party/WebKit/Source/core/style/ClipPathOperation.cpp
diff --git a/third_party/WebKit/Source/core/style/ClipPathOperation.cpp b/third_party/WebKit/Source/core/style/ClipPathOperation.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..0ea721d510ec51c722779f0cafe8e1c7b5958ee7
--- /dev/null
+++ b/third_party/WebKit/Source/core/style/ClipPathOperation.cpp
@@ -0,0 +1,24 @@
+// Copyright 2016 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/style/ClipPathOperation.h"
+
+namespace blink {
+
+void ReferenceClipPathOperation::addClient(SVGResourceClient* client) {
+  m_elementProxy->addClient(client);

[chrishtr, 2016/11/14 20:56:13]

Why have these helper methods if elementProxy() is also available as a method on
this
class?

[fs, 2016/11/15 14:10:56]

I guess I thought it looked prettier... (I did this for the corresponding filter
case too.) Maybe I should delegate findElement() too for consistency then?

[chrishtr, 2016/11/15 16:24:22]

Maybe. I don't feel that strongly about it or my original comment, up to you.

[fs, 2016/11/15 16:29:32]

Noted. I've dropped the accessor (elementProxy()) and added a delegation to
findElement too.

+}
+
+void ReferenceClipPathOperation::removeClient(SVGResourceClient* client) {
+  m_elementProxy->removeClient(client);
+}
+
+bool ReferenceClipPathOperation::operator==(const ClipPathOperation& o) const {
+  if (!isSameType(o))
+    return false;
+  const ReferenceClipPathOperation& other = toReferenceClipPathOperation(o);
+  return m_elementProxy == other.m_elementProxy && m_url == other.m_url;
+}
+
+}  // namespace blink