Improve PasswordProtectionService and PasswordProtectionRequest

components/safe_browsing/csd.proto

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This proto file includes:
 // (1) Client side phishing and malware detection request and response
 //   protocol buffers.  Those protocol messages should be kept in sync
 //   with the server implementation.
 //
 // (2) Safe Browsing reporting protocol buffers.
@@ skipping 232 matching lines @@
   //    www.google.com/foo/bar?param=val -> google.com
   //    www.geocities.com/foo/bar.html -> geocities.com/foo
   //    adwords.blogspot.com/index.html -> adwords.blogspot.com
   //
   // The pattern will always match the page_url of the request, and will be
   // a substring of page_url.
   optional string cache_expression = 3;
 
   // Deprecated.
   optional bool DEPRECATED_cache_expression_exact_match = 4 [deprecated = true];
+
+  // A token which is consistent in the response and post-warning actions.

[lpz, 2017/04/18 14:43:27]

So this field is not used in chrome code yet, right? Are there plans to? Might
seem strange to have a field that is never accessed

[Jialiu Lin, 2017/04/18 20:38:03]

This field is going to be used in the post-incident reporting. I'll move it to
later CL when it is actually being used. 

+  optional bytes verdict_token = 5;
 }
 
 message ClientMalwareResponse {
   required bool blacklist = 1;
   // The confirmed blacklisted bad IP and its url, which will be shown in
   // malware warning, if the blacklist verdict is true.
   // This IP string could be either in IPv4 or IPv6 format, which is the same
   // as the ones client sent to server.
   optional string bad_ip = 2;
   optional string bad_url = 3;
@@ skipping 760 matching lines @@
 // There is no response (an empty body) to this request.
 message NotificationImageReportRequest {
   optional string notification_origin = 1;  // Src-origin of the notification.
   optional ImageData image = 2;             // The bitmap of the image.
 
   // Note that the image URL is deliberately omitted as it would be untrusted,
   // since the notification image fetch may be intercepted by a Service Worker
   // (even if the image URL is cross-origin). Otherwise a website could mislead
   // Safe Browsing into associating phishing image bitmaps with safe image URLs.
 }