Add management related code to SafeBrowsingNavigationObserverManager

chrome/common/safe_browsing/csd.proto

Index: chrome/common/safe_browsing/csd.proto
diff --git a/chrome/common/safe_browsing/csd.proto b/chrome/common/safe_browsing/csd.proto
index ce779caed7efee915159bd625a6394ceddf7bcde..c09f83a25f3e4d0da4a2f039664e2f16172bcd8e 100644
--- a/chrome/common/safe_browsing/csd.proto
+++ b/chrome/common/safe_browsing/csd.proto
@@ -382,14 +382,13 @@ message ClientDownloadRequest {
   // the leading extension separator.
   repeated string alternate_extensions = 35;
 
-  message URLChainEntry {
+  message ReferrerChainEntry {

[Nathan Parker, 2016/12/05 22:21:24]

This doesn't look like a backward-compatible change.  Is that the intention?

Another idea: we could add "ReferrerChainEntry referrer_chain_entry=37" and keep
"URLChainEntry urlchain=36" and continue to populate the latter until the
backend code no longer looks at it.

Or just ensure it's backward compatible, and plan to do a big (mechanical)
renaming of things in Google3 so it matches.

[Jialiu Lin, 2016/12/06 23:10:06]

So far, backend is not using field 36. So I guess it is still safe to make these
changes. 
I'll make sure google3 proto updated before I land this CL. 

[Nathan Parker, 2016/12/06 23:31:32]

Ah great then if it's not being used there's no issue.

     enum URLType {
       DOWNLOAD_URL = 1;
-      DOWNLOAD_REFERRER = 2;
-      LANDING_PAGE = 3;
-      LANDING_REFERRER = 4;
-      CLIENT_REDIRECT = 5;
-      SERVER_REDIRECT = 6;
+      LANDING_PAGE = 2;
+      LANDING_REFERRER = 3;
+      CLIENT_REDIRECT = 4;
+      SERVER_REDIRECT = 5;
     }
 
     // [required] The url of this Entry.
@@ -398,28 +397,25 @@ message ClientDownloadRequest {
     // Type of URLs, such as download url, download referrer, etc.
     optional URLType type = 2;
 
-    // IP address corresponding to url.
-    optional string ip_address = 3;
+    // IP addresses corresponding to this host.
+    repeated string ip_address = 3;
 
     // Referrer url of this entry.
-    optional string referrer = 4;
+    optional string referrer_url = 4;
 
     // Main frame URL of referrer.
-    optional string main_frame_referrer = 5;
+    optional string referrer_main_frame_url = 5;
 
     // If this URL loads in a different tab/frame from previous one.
     optional bool is_retargeting = 6;
 
-    // If there is a user gesture attached to this transition.
-    optional bool is_user_initiated = 7;
-
-    optional double timestamp_in_millisec = 8;
+    optional double navigation_time_msec = 7;
   }  // End of URLChainEntry
 
   // URLs transitions from landing referrer to download in reverse chronological
   // order, i.e. download url comes first in this list, and landing referrer
   // comes last.
-  repeated URLChainEntry url_chain = 36;
+  repeated ReferrerChainEntry referrer_chain_entry = 36;
 }
 
 message ClientDownloadResponse {