Add missing data members in StyleRareNonInheritedData copy constructor.

third_party/WebKit/Source/core/style/StyleRareNonInheritedData.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 56 matching lines @@
 // Page size type.
 // StyleRareNonInheritedData::page_size_ is meaningful only when
 // StyleRareNonInheritedData::page_size_type_ is PAGE_SIZE_RESOLVED.
 enum PageSizeType {
   PAGE_SIZE_AUTO,            // size: auto
   PAGE_SIZE_AUTO_LANDSCAPE,  // size: landscape
   PAGE_SIZE_AUTO_PORTRAIT,   // size: portrait
   PAGE_SIZE_RESOLVED         // Size is fully resolved.
 };
 
+using PaintImages = Vector<Persistent<StyleImage>>;

[meade_UTC10, 2017/05/31 05:59:45]

Do you expect PaintImages to be used by anything outside of
StyleRareNonInheritedData? If not, you probably shouldn't create an alias.

https://google.github.io/styleguide/cppguide.html#Aliases

[shend, 2017/05/31 06:58:25]

Done.

+
 // This struct is for rarely used non-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 StyleRareNonInheritedData
     : public RefCounted<StyleRareNonInheritedData> {
  public:
   static PassRefPtr<StyleRareNonInheritedData> Create() {
@@ skipping 75 matching lines @@
   StyleColor visited_link_text_decoration_color_;
   StyleColor visited_link_background_color_;
   StyleColor visited_link_outline_color_;
   StyleColor visited_link_border_left_color_;
   StyleColor visited_link_border_right_color_;
   StyleColor visited_link_border_top_color_;
   StyleColor visited_link_border_bottom_color_;
 
   Vector<String> callback_selectors_;
 
-  std::unique_ptr<Vector<Persistent<StyleImage>>> paint_images_;
+  std::unique_ptr<PaintImages> paint_images_;
 
   std::unique_ptr<StyleNonInheritedVariables> variables_;
 
   StyleContentAlignmentData align_content_;
   StyleSelfAlignmentData align_items_;
   StyleSelfAlignmentData align_self_;
   StyleContentAlignmentData justify_content_;
   StyleSelfAlignmentData justify_items_;
   StyleSelfAlignmentData justify_self_;
 
@@ skipping 57 matching lines @@
   unsigned has_author_border_ : 1;  // Whether there is a author-defined border.
 
  private:
   StyleRareNonInheritedData();
   StyleRareNonInheritedData(const StyleRareNonInheritedData&);
 };
 
 }  // namespace blink
 
 #endif  // StyleRareNonInheritedData_h