Precache should cancel when there is user traffic

components/precache/core/precache_fetcher.cc

Index: components/precache/core/precache_fetcher.cc
diff --git a/components/precache/core/precache_fetcher.cc b/components/precache/core/precache_fetcher.cc
index dc0d3a272327a9ce1002ba33b377b2abb2f7e058..1cfb7770c4f002446a42c2b3755e40605c36a9c3 100644
--- a/components/precache/core/precache_fetcher.cc
+++ b/components/precache/core/precache_fetcher.cc
@@ -300,7 +300,7 @@ void PrecacheFetcher::RecordCompletionStatistics(
 
   // If there are resource URLs left to fetch, the last manifest is not yet
   // completed.
-  if (remaining_resource_urls_to_fetch > 0)
+  if (remaining_resource_urls_to_fetch > 0 && manifests_completed > 0)

[bengr, 2016/07/06 17:04:27]

This seems like it should be a DCHECK. Could you explain why it is ok for this
method to be called when manifests_completed <= 0?

[Raj, 2016/07/06 18:28:23]

hmm. I got this failure in a few of my tests.
Now it is less apparent to me why this would be the case.
I removed this check. Will fix it when I hit such issue again.

     --manifests_completed;
 
   DCHECK_GE(manifests_completed, 0);