Adding NetLog support to SafeBrowsingPingManager.

net/log/net_log_event_type_list.h

Index: net/log/net_log_event_type_list.h
diff --git a/net/log/net_log_event_type_list.h b/net/log/net_log_event_type_list.h
index f0d43fff931eb22fcdde38bf927e630020ef2bda..d5ebee6c6f30585f52796c5d60938e6e76396ef4 100644
--- a/net/log/net_log_event_type_list.h
+++ b/net/log/net_log_event_type_list.h
@@ -3020,6 +3020,23 @@ EVENT_TYPE(SAFE_BROWSING_CHECKING_URL)
 //  }
 EVENT_TYPE(SAFE_BROWSING_DEFERRED)
 
+// The start/end of a Safe Browsing ping being sent.
+//
+// The BEGIN phase contains the following parameters:
+//  {
+//    "url": <The URL the ping is going to, which identifies the type of ping
+//            that is being sent (eg: ThreatReport, SafeBrowsingHit)>
+//    "data": <The base64 encoding of the payload sent with the ping>
+//
+// The END phase contains the following parameters:
+//  {
+//    "url": <The URL the ping is going to, which identifies the type of ping
+//            that is being sent (eg: ThreatReport, SafeBrowsingHit)>

[mmenke, 2016/10/03 15:11:46]

Is this completely redundant with the corresponding begin phase event?  If so,
should probably remove it.

[lpz, 2016/10/04 18:48:32]

Yes, redundant. Removed.

+//    "status": <The integer status of the report transmission. Corresponds to
+//               URLRequestStatus::Status>
+//    "error": <The error code returned by the server, 0 indicating success>
+EVENT_TYPE(SAFE_BROWSING_PING)
+
 // Marks start of UploadDataStream that is logged on initialization.
 // The END phase contains the following parameters:
 // {