Add danakj as OWNERS for content/common/cc_messages_unittest.cc

content/common/OWNERS

Index: content/common/OWNERS
diff --git a/content/common/OWNERS b/content/common/OWNERS
index fd688b22610258b5c14732ec5a0d9de05d1b6166..706e79addc106ac4b08fe12736ad7159010bdd64 100644
--- a/content/common/OWNERS
+++ b/content/common/OWNERS
@@ -50,3 +50,6 @@ per-file accessibility_node_data.*=dtseng@chromium.org
 per-file websocket.*=ricea@chromium.org
 per-file websocket.*=tyoshino@chromium.org
 per-file websocket.*=yhirano@chromium.org
+
+# Compositor
+per-file cc_messages*=danakj@chromium.org

[jam, 2014/06/18 17:14:58]

all IPC message files need a person from the security review to sign off on (see
the comment on line 21).

[danakj, 2014/06/18 17:28:00]

I was mimicing the WebSocket above. I can change this to .cc and _unittest.cc if
you prefer? Was not trying to sidestep the security team rules on the .h, I
assumed this worked out from the WebSocket rule, I guess not?

[jam, 2014/06/18 18:04:16]

The websocket exception is for websocket.*, which won't cover
websocket_messages.h (i.e. security team still has to review that IPC file)

cc_messages.cc has the IPC param traits, which is also something that security
team wants to review.
no problem, I didn't mean to imply that you're trying to sidestep that.

[danakj, 2014/06/18 18:08:21]

Is it? There is no rule for that today. We have to change the header file to add
a new thing to a message.