Have a list of pending checks instead of pending clients

chrome/browser/loader/safe_browsing_resource_throttle.h

Index: chrome/browser/loader/safe_browsing_resource_throttle.h
diff --git a/chrome/browser/loader/safe_browsing_resource_throttle.h b/chrome/browser/loader/safe_browsing_resource_throttle.h
index 8a821412ed244b1b57b1b37f803a83bde5056262..81eed113d1e1061fbf9a9bf86792438a6a4d820d 100644
--- a/chrome/browser/loader/safe_browsing_resource_throttle.h
+++ b/chrome/browser/loader/safe_browsing_resource_throttle.h
@@ -168,6 +168,11 @@ class SafeBrowsingResourceThrottle
   GURL unchecked_redirect_url_;
   GURL url_being_checked_;
 
+  // TODO(vakh): The following vector should be removed after fixing
+  // http://crbug.com/660293
+  // URLs that timed out waiting for a SafeBrowsing reputation check.
+  std::vector<GURL> timed_out_urls_;

[Charlie Harrison, 2017/01/06 16:53:00]

Do we really need the GURL copies here?

The URLRequest already stores its redirect chain, and we store a copy of it
above too (is that really necessary?).

[vakh (use Gerrit instead), 2017/01/09 19:21:32]

You mean just store references to those GURLs?
For a couple of reasons:
1. It isn't clear, to me, if the SafeBrowsingResourceThrottle's lifetime is the
same as, or is greater than the URLRequest.
2. The vector will only ever be populated if the SafeBrowsing check times out.
This isn't a common occurrence.
3. This code is being added for diagnostic purposes and will be removed shortly
afterwards so optimizing it has very limited value.

[Charlie Harrison, 2017/01/09 21:46:53]

A resource throttle is owned by a ThrottlingResourceHandler. Resource handler
has the following class comment:
"A ResourceHandler's lifetime is bound to its associated URLRequest". So I think
we should be fine here.
Ack.
As long as this is true, LGTM. I would be very interested in removing the
redirect_urls_ vector as well in the future though. URLs can be big (up to 2MB
each), so adding more places where we log redirects in the browser process makes
me nervous.

+
   scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager> database_manager_;
   scoped_refptr<safe_browsing::SafeBrowsingUIManager> ui_manager_;
   const net::URLRequest* request_;