Report downloaded resources at most once

components/precache/core/precache_database.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_PRECACHE_CORE_PRECACHE_DATABASE_H_
 #define COMPONENTS_PRECACHE_CORE_PRECACHE_DATABASE_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 74 matching lines @@
                             const base::TimeDelta& latency,
                             const base::Time& fetch_time,
                             const net::HttpResponseInfo& info,
                             int64_t size,
                             int host_rank,
                             bool is_connection_cellular);
 
   // Returns the referrer host entry for the |referrer_host|.
   PrecacheReferrerHostEntry GetReferrerHost(const std::string& referrer_host);
 
-  // Populates the list of used and unused resources for referrer host with id
-  // |referrer_host_id|.
+  // Populates the list of used and downloaded resources for referrer host with
+  // id |referrer_host_id|. It will also clear the reported downloaded_urls.
   void GetURLListForReferrerHost(int64_t referrer_host_id,
                                  std::vector<GURL>* used_urls,
-                                 std::vector<GURL>* unused_urls);
+                                 std::vector<GURL>* downloaded_urls);
 
   // Updates the |manifest_id| and |fetch_time| for the referrer host
   // |hostname|, and deletes the precached subresource URLs for this top host.
   void UpdatePrecacheReferrerHost(const std::string& hostname,
                                   int64_t manifest_id,
                                   const base::Time& fetch_time);
 
   // Gets the state required to continue a precache session.
   std::unique_ptr<PrecacheUnfinishedWork> GetUnfinishedWork();
 
   // Stores the state required to continue a precache session so that the
   // session can be resumed later.
   void SaveUnfinishedWork(
       std::unique_ptr<PrecacheUnfinishedWork> unfinished_work);
 
   // Deletes unfinished work from the database.
   void DeleteUnfinishedWork();
 
   // Precache quota.
   void SaveQuota(const PrecacheQuota& quota);
   PrecacheQuota GetQuota();
 
   base::WeakPtr<PrecacheDatabase> GetWeakPtr();
 
  private:
   friend class PrecacheDatabaseTest;
+  friend class PrecacheFetcherTest;
 
   bool IsDatabaseAccessible() const;
 
   // Flushes any buffered write operations. |buffered_writes_| will be empty
   // after calling this function. To maximize performance, all the buffered
   // writes are run in a single database transaction.
   void Flush();
 
   // Same as Flush(), but also updates the flag |is_flush_posted_| to indicate
   // that a flush is no longer posted.
   void PostedFlush();
 
   // Post a call to PostedFlush() on the current thread's MessageLoop, if
   // |buffered_writes_| is non-empty and there isn't already a flush call
   // posted.
   void MaybePostFlush();
 
   // Records the time since the last precache.
   void RecordTimeSinceLastPrecache(const base::Time& fetch_time);
 
   void RecordURLPrefetchInternal(const GURL& url,
                                  const std::string& referrer_host,
                                  bool is_precached,
-                                 const base::Time& fetch_time);
+                                 const base::Time& fetch_time,
+                                 bool is_download_reported);
 
   void UpdatePrecacheReferrerHostInternal(const std::string& hostname,
                                           int64_t manifest_id,
                                           const base::Time& fetch_time);
 
   std::unique_ptr<sql::Connection> db_;
 
   // Table that keeps track of URLs that are in the cache because of precaching,
   // and wouldn't be in the cache otherwise. If |buffered_writes_| is non-empty,
   // then this table will not be up to date until the next call to Flush().
@@ skipping 28 matching lines @@
 
   // This must be the last member of this class.
   base::WeakPtrFactory<PrecacheDatabase> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PrecacheDatabase);
 };
 
 }  // namespace precache
 
 #endif  // COMPONENTS_PRECACHE_CORE_PRECACHE_DATABASE_H_