Add more out of line copy ctors for complex classes.

This patch adds the remaining copy constructors for complex classes.
After this patch, it should be possible to enable the heavy class copy
constructor checks by default.

R=thakis@chromium.org, dcheng@chromium.org
TBR=jam@chromium.org
BUG=436357
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review URL: https://codereview.chromium.org/1825273002

Cr-Commit-Position: refs/heads/master@{#383131}

2 files changed, 4 insertions, 0 deletions

diff --git a/cc/surfaces/surface_manager.cc b/cc/surfaces/surface_manager.cc
index 23b49bc..d22d1e5 100644
--- a/cc/surfaces/surface_manager.cc
+++ b/cc/surfaces/surface_manager.cc
@@ -17,6 +17,9 @@ namespace cc {
 SurfaceManager::ClientSourceMapping::ClientSourceMapping()
     : client(nullptr), source(nullptr) {}
 
+SurfaceManager::ClientSourceMapping::ClientSourceMapping(
+    const ClientSourceMapping& other) = default;
+
 SurfaceManager::ClientSourceMapping::~ClientSourceMapping() {
   DCHECK(is_empty()) << "client: " << client
                      << ", children: " << children.size();
diff --git a/cc/surfaces/surface_manager.h b/cc/surfaces/surface_manager.h
index 8ddace4..55e44f8 100644
--- a/cc/surfaces/surface_manager.h
+++ b/cc/surfaces/surface_manager.h
@@ -126,6 +126,7 @@ class CC_SURFACES_EXPORT SurfaceManager {
   // pointers guaranteed alive by callers until unregistered.
   struct ClientSourceMapping {
     ClientSourceMapping();
+    ClientSourceMapping(const ClientSourceMapping& other);
     ~ClientSourceMapping();
     bool is_empty() const { return !client && !children.size(); }
     // The client that's responsible for creating this namespace.  Never null.