Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(127)

Side by Side Diff: chrome/common/safe_browsing/csd.proto

Issue 2686773002: In previous implementation, if the server does the more than one server redirects, we only record t… (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This proto file includes: 5 // This proto file includes:
6 // (1) Client side phishing and malware detection request and response 6 // (1) Client side phishing and malware detection request and response
7 // protocol buffers. Those protocol messages should be kept in sync 7 // protocol buffers. Those protocol messages should be kept in sync
8 // with the server implementation. 8 // with the server implementation.
9 // 9 //
10 // (2) Safe Browsing reporting protocol buffers. 10 // (2) Safe Browsing reporting protocol buffers.
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
379 379
380 // PPAPI_SAVE_REQUEST type messages may have more than one suggested filetype. 380 // PPAPI_SAVE_REQUEST type messages may have more than one suggested filetype.
381 // Each element in this collection indicates an alternate extension including 381 // Each element in this collection indicates an alternate extension including
382 // the leading extension separator. 382 // the leading extension separator.
383 repeated string alternate_extensions = 35; 383 repeated string alternate_extensions = 35;
384 384
385 // URLs transitions from landing referrer to download in reverse chronological 385 // URLs transitions from landing referrer to download in reverse chronological
386 // order, i.e. download url comes first in this list, and landing referrer 386 // order, i.e. download url comes first in this list, and landing referrer
387 // comes last. 387 // comes last.
388 repeated ReferrerChainEntry referrer_chain = 36; 388 repeated ReferrerChainEntry referrer_chain = 36;
389
390 // Whether DownloadAttribution Finch experiment is enabled for this ping.
391 optional bool download_attribution_finch_enabled = 39;
389 } 392 }
390 393
391 message ReferrerChainEntry { 394 message ReferrerChainEntry {
392 enum URLType { 395 enum URLType {
393 DOWNLOAD_URL = 1; 396 DOWNLOAD_URL = 1;
394 LANDING_PAGE = 2; 397 LANDING_PAGE = 2;
395 LANDING_REFERRER = 3; 398 LANDING_REFERRER = 3;
396 CLIENT_REDIRECT = 4; 399 CLIENT_REDIRECT = 4;
397 SERVER_REDIRECT = 5; 400 DEPRECATED_SERVER_REDIRECT = 5; // Deprecated
401 }
402
403 message ServerRedirect {
404 // [required] server redirect url
405 optional string url = 1;
406
407 // Additional fields for future expansion.
398 } 408 }
399 409
400 // [required] The url of this Entry. 410 // [required] The url of this Entry.
401 optional string url = 1; 411 optional string url = 1;
402 412
413 // Only set if it is different from |url|.
414 optional string main_frame_url = 9;
415
403 // Type of URLs, such as download url, download referrer, etc. 416 // Type of URLs, such as download url, download referrer, etc.
404 optional URLType type = 2; 417 optional URLType type = 2 [default = CLIENT_REDIRECT];
405 418
406 // IP addresses corresponding to this host. 419 // IP addresses corresponding to this host.
407 repeated string ip_addresses = 3; 420 repeated string ip_addresses = 3;
408 421
409 // Referrer url of this entry. 422 // Referrer url of this entry.
410 optional string referrer_url = 4; 423 optional string referrer_url = 4;
411 424
412 // Main frame URL of referrer. 425 // Main frame URL of referrer.
426 // Only set if it is different from |referrer_url|.
413 optional string referrer_main_frame_url = 5; 427 optional string referrer_main_frame_url = 5;
414 428
415 // If this URL loads in a different tab/frame from previous one. 429 // If this URL loads in a different tab/frame from previous one.
416 optional bool is_retargeting = 6; 430 optional bool is_retargeting = 6;
417 431
418 optional double navigation_time_msec = 7; 432 optional double navigation_time_msec = 7;
419 } // End of URLChainEntry 433
434 // Set only if server redirects happened in navigation.
435 // The first entry in |server_redirect_chain| should be the original request
436 // url, and the last entry should be the same as |url|.
437 repeated ServerRedirect server_redirect_chain = 8;
438 } // End of ReferrerChainEntry
420 439
421 message ClientDownloadResponse { 440 message ClientDownloadResponse {
422 enum Verdict { 441 enum Verdict {
423 // Download is considered safe. 442 // Download is considered safe.
424 SAFE = 0; 443 SAFE = 0;
425 // Download is considered dangerous. Chrome should show a warning to the 444 // Download is considered dangerous. Chrome should show a warning to the
426 // user. 445 // user.
427 DANGEROUS = 1; 446 DANGEROUS = 1;
428 // Download is uncommon. Chrome should display a less severe warning. 447 // Download is uncommon. Chrome should display a less severe warning.
429 UNCOMMON = 2; 448 UNCOMMON = 2;
(...skipping 428 matching lines...) Expand 10 before | Expand all | Expand 10 after
858 // There is no response (an empty body) to this request. 877 // There is no response (an empty body) to this request.
859 message NotificationImageReportRequest { 878 message NotificationImageReportRequest {
860 optional string notification_origin = 1; // Src-origin of the notification. 879 optional string notification_origin = 1; // Src-origin of the notification.
861 optional ImageData image = 2; // The bitmap of the image. 880 optional ImageData image = 2; // The bitmap of the image.
862 881
863 // Note that the image URL is deliberately omitted as it would be untrusted, 882 // Note that the image URL is deliberately omitted as it would be untrusted,
864 // since the notification image fetch may be intercepted by a Service Worker 883 // since the notification image fetch may be intercepted by a Service Worker
865 // (even if the image URL is cross-origin). Otherwise a website could mislead 884 // (even if the image URL is cross-origin). Otherwise a website could mislead
866 // Safe Browsing into associating phishing image bitmaps with safe image URLs. 885 // Safe Browsing into associating phishing image bitmaps with safe image URLs.
867 } 886 }
OLDNEW
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_navigation_observer_unittest.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698