Commit 3f01e1e8 authored by Abhishek Arya's avatar Abhishek Arya Committed by Commit Bot

Log file resource dependencies for ClusterFuzz.

ClusterFuzz relied on NetworkDelegate::NotifyBeforeURLRequest to
know a test case's resource dependencies and archive them. This
regressed when file url loads were moved to FileURLLoader. Add
logging to capture them for file loads.

Bug: 715656
Change-Id: Ibb2938a569223534d2b7dc7d757390973a6d172e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1845589Reviewed-by: default avatarMatt Menke <mmenke@chromium.org>
Commit-Queue: Abhishek Arya <inferno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703826}
parent f4a5045f
...@@ -436,6 +436,10 @@ class FileURLLoader : public network::mojom::URLLoader { ...@@ -436,6 +436,10 @@ class FileURLLoader : public network::mojom::URLLoader {
LinkFollowingPolicy link_following_policy, LinkFollowingPolicy link_following_policy,
std::unique_ptr<FileURLLoaderObserver> observer, std::unique_ptr<FileURLLoaderObserver> observer,
scoped_refptr<net::HttpResponseHeaders> extra_response_headers) { scoped_refptr<net::HttpResponseHeaders> extra_response_headers) {
// ClusterFuzz depends on the following VLOG to get resource dependencies.
// See crbug.com/715656.
VLOG(1) << "FileURLLoader::Start: " << request.url;
network::ResourceResponseHead head; network::ResourceResponseHead head;
head.request_start = base::TimeTicks::Now(); head.request_start = base::TimeTicks::Now();
head.response_start = base::TimeTicks::Now(); head.response_start = base::TimeTicks::Now();
......
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