Commit 497d3fb5 authored by Varun Khaneja's avatar Varun Khaneja Committed by Commit Bot

Set page_url in CSBRR proto sent on APK downloads

Also, add a comment in the proto.
See http://b/123719599 for details.

R=drubery

Bug: 907280,927831
Change-Id: Idc690f25bfc3b909eead54c0184184bcf0df8a32
Reviewed-on: https://chromium-review.googlesource.com/c/1450415Reviewed-by: default avatarDaniel Rubery <drubery@chromium.org>
Commit-Queue: Varun Khaneja <vakh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#628425}
parent 260184f4
......@@ -259,6 +259,7 @@ void AndroidTelemetryService::MaybeSendApkDownloadReport(
report->set_type(
safe_browsing::ClientSafeBrowsingReportRequest::APK_DOWNLOAD);
report->set_url(item->GetOriginalUrl().spec());
report->set_page_url(item->GetTabUrl().spec());
// Fill referrer chain.
content::WebContents* web_contents =
......
......@@ -1180,7 +1180,12 @@ message ClientSafeBrowsingReportRequest {
// URL of the page in the address bar.
optional string url = 1;
// Must be set if the ReportType is not one of DANGEROUS_DOWNLOAD_RECOVERY,
// DANGEROUS_DOWNLOAD_WARNING, DANGEROUS_DOWNLOAD_BY_API,
// DANGEROUS_DOWNLOAD_OPENED, URL_PASSWORD_PROTECTION_PHISHING.
optional string page_url = 2;
optional string referrer_url = 3;
repeated Resource resources = 4;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment