Cleanup StyleRareInheritedData.

third_party/WebKit/Source/core/style/StyleRareInheritedData.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 10 matching lines @@
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #ifndef StyleRareInheritedData_h
 #define StyleRareInheritedData_h
 
 #include "core/CoreExport.h"
 #include "core/css/StyleAutoColor.h"
 #include "core/css/StyleColor.h"
+#include "core/style/AppliedTextDecoration.h"
 #include "core/style/AppliedTextDecorationList.h"
+#include "core/style/CursorData.h"
 #include "core/style/CursorList.h"
+#include "core/style/QuotesData.h"
+#include "core/style/ShadowData.h"
+#include "core/style/ShadowList.h"
+#include "core/style/StyleImage.h"
+#include "core/style/StyleInheritedVariables.h"
 #include "core/style/TextSizeAdjust.h"
 #include "platform/Length.h"
 #include "platform/graphics/Color.h"
 #include "platform/heap/Handle.h"
 #include "platform/text/TabSize.h"
 #include "platform/wtf/PassRefPtr.h"
 #include "platform/wtf/RefCounted.h"
 #include "platform/wtf/RefVector.h"
 #include "platform/wtf/text/AtomicString.h"
 
 namespace blink {
 
 class QuotesData;
 class ShadowList;
 class StyleImage;
 class StyleInheritedVariables;

[alancutter (OOO until 2018), 2017/05/16 06:23:37]

These need removing.

 
 // This struct is for rarely used inherited CSS3, CSS2, and WebKit-specific
 // properties.  By grouping them together, we save space, and only allocate this
 // object when someone actually uses one of these properties.
 // TODO(sashab): Move this into a private class on ComputedStyle, and remove
 // all methods on it, merging them into copy/creation methods on ComputedStyle
 // instead. Keep the allocation logic, only allocating a new object if needed.
 class CORE_EXPORT StyleRareInheritedData
-    : public RefCounted<StyleRareInheritedData> {
+    : public RefCountedCopyable<StyleRareInheritedData> {
  public:
   static PassRefPtr<StyleRareInheritedData> Create() {
     return AdoptRef(new StyleRareInheritedData);
   }
   PassRefPtr<StyleRareInheritedData> Copy() const {
     return AdoptRef(new StyleRareInheritedData(*this));
   }
-  ~StyleRareInheritedData();
 
   bool operator==(const StyleRareInheritedData&) const;
   bool operator!=(const StyleRareInheritedData& o) const {
     return !(*this == o);
   }
   bool ShadowDataEquivalent(const StyleRareInheritedData&) const;
   bool QuotesDataEquivalent(const StyleRareInheritedData&) const;
 
   Persistent<StyleImage> list_style_image_;
 
@@ skipping 81 matching lines @@
   Color tap_highlight_color_;
 
   RefPtr<AppliedTextDecorationList> applied_text_decorations_;
   TabSize tab_size_;
 
   RefPtr<StyleInheritedVariables> variables_;
   TextSizeAdjust text_size_adjust_;
 
  private:
   StyleRareInheritedData();
-  StyleRareInheritedData(const StyleRareInheritedData&);
+  StyleRareInheritedData(const StyleRareInheritedData&) = default;

[alancutter (OOO until 2018), 2017/05/16 06:24:35]

Can = default be used in CPP files? Perhaps it's better in the interest of
incremental build times to keep the includes to CPP files where possible.

 };
 
 }  // namespace blink
 
 #endif  // StyleRareInheritedData_h