summaryrefslogtreecommitdiffstats
path: root/remoting/signaling
diff options
context:
space:
mode:
authorvmpstr <vmpstr@chromium.org>2016-02-26 13:46:54 -0800
committerCommit bot <commit-bot@chromium.org>2016-02-26 21:49:14 +0000
commit83a7f26e6402938c9afe9c68d4a0bd82791614e8 (patch)
tree0de13568952b09edadd2507c8d395af3e4153c8e /remoting/signaling
parente4dd8cc726ed956556807025eb6808db2cd8b8cd (diff)
downloadchromium_src-83a7f26e6402938c9afe9c68d4a0bd82791614e8.zip
chromium_src-83a7f26e6402938c9afe9c68d4a0bd82791614e8.tar.gz
chromium_src-83a7f26e6402938c9afe9c68d4a0bd82791614e8.tar.bz2
remoting: Add out-of-line copy ctors for complex classes.
This patch adds out of line copy constructors for classes that our clang-plugin considers heavy. This is an effort to enable copy constructor checks by default. BUG=436357 R=garykac@chromium.org, dcheng@chromium.org, thakis@chromium.org Review URL: https://codereview.chromium.org/1736633002 Cr-Commit-Position: refs/heads/master@{#377988}
Diffstat (limited to 'remoting/signaling')
-rw-r--r--remoting/signaling/server_log_entry.cc2
-rw-r--r--remoting/signaling/server_log_entry.h1
-rw-r--r--remoting/signaling/xmpp_signal_strategy.cc3
-rw-r--r--remoting/signaling/xmpp_signal_strategy.h1
4 files changed, 7 insertions, 0 deletions
diff --git a/remoting/signaling/server_log_entry.cc b/remoting/signaling/server_log_entry.cc
index 9eda8931..080f8ea 100644
--- a/remoting/signaling/server_log_entry.cc
+++ b/remoting/signaling/server_log_entry.cc
@@ -35,6 +35,8 @@ const char kKeyCpu[] = "cpu";
ServerLogEntry::ServerLogEntry() {
}
+ServerLogEntry::ServerLogEntry(const ServerLogEntry& other) = default;
+
ServerLogEntry::~ServerLogEntry() {
}
diff --git a/remoting/signaling/server_log_entry.h b/remoting/signaling/server_log_entry.h
index f1b0ece..fbcba0f 100644
--- a/remoting/signaling/server_log_entry.h
+++ b/remoting/signaling/server_log_entry.h
@@ -28,6 +28,7 @@ class ServerLogEntry {
};
ServerLogEntry();
+ ServerLogEntry(const ServerLogEntry& other);
~ServerLogEntry();
// Sets an arbitrary key/value entry.
diff --git a/remoting/signaling/xmpp_signal_strategy.cc b/remoting/signaling/xmpp_signal_strategy.cc
index fbc5373..10aea02 100644
--- a/remoting/signaling/xmpp_signal_strategy.cc
+++ b/remoting/signaling/xmpp_signal_strategy.cc
@@ -46,6 +46,9 @@ XmppSignalStrategy::XmppServerConfig::XmppServerConfig()
: port(kDefaultXmppPort), use_tls(true) {
}
+XmppSignalStrategy::XmppServerConfig::XmppServerConfig(
+ const XmppServerConfig& other) = default;
+
XmppSignalStrategy::XmppServerConfig::~XmppServerConfig() {
}
diff --git a/remoting/signaling/xmpp_signal_strategy.h b/remoting/signaling/xmpp_signal_strategy.h
index 8878547..4743d9f 100644
--- a/remoting/signaling/xmpp_signal_strategy.h
+++ b/remoting/signaling/xmpp_signal_strategy.h
@@ -25,6 +25,7 @@ class XmppSignalStrategy : public SignalStrategy {
// XMPP Server configuration for XmppSignalStrategy.
struct XmppServerConfig {
XmppServerConfig();
+ XmppServerConfig(const XmppServerConfig& other);
~XmppServerConfig();
std::string host;