Add more out of line copy ctors for complex classes.

content/public/common/frame_navigate_params.h

Index: content/public/common/frame_navigate_params.h
diff --git a/content/public/common/frame_navigate_params.h b/content/public/common/frame_navigate_params.h
index 3b8dd90d25d38276b3261c00f3878e5cf1a705cb..b56ab21c842953f2d3fb10bc395d6be859d063b4 100644
--- a/content/public/common/frame_navigate_params.h
+++ b/content/public/common/frame_navigate_params.h
@@ -12,6 +12,7 @@
 
 #include "content/common/content_export.h"
 #include "content/public/common/referrer.h"
+#include "ipc/ipc_message_utils.h"
 #include "net/base/host_port_pair.h"
 #include "ui/base/page_transition_types.h"
 #include "url/gurl.h"
@@ -19,7 +20,7 @@
 namespace content {
 
 // Struct used by WebContentsObserver.
-struct CONTENT_EXPORT FrameNavigateParams {
+struct CONTENT_EXPORT FrameNavigateParams : public IPC::NoParams {

[vmpstr, 2016/03/22 23:22:43]

Note that this is required, because there is an IPC struct that derives from
content::FrameNagivateParams and then is copied. The clang plugin check skips
any struct that is derived (directly or indirectly) from IPC::NoParams. This
seemed like the easiest way to fix the issue.

Let me know if you have a different solution in mind. 

[dcheng, 2016/03/22 23:25:06]

It's definitely kind of a hack, but I think it's OK. The alternatives seem
strictly worse.

[no sievers, 2016/03/22 23:37:20]

I'd put a comment though.

[vmpstr, 2016/03/22 23:57:43]

Done.

   FrameNavigateParams();
   FrameNavigateParams(const FrameNavigateParams& other);
   ~FrameNavigateParams();