Precache tracking database

components/precache/core/precache_database.h

Index: components/precache/core/precache_database.h
diff --git a/components/precache/core/precache_database.h b/components/precache/core/precache_database.h
new file mode 100644
index 0000000000000000000000000000000000000000..43e9ea2dc6158fa48062021231b7a218ec5a4af0
--- /dev/null
+++ b/components/precache/core/precache_database.h
@@ -0,0 +1,63 @@
+// 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 "base/basictypes.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/threading/non_thread_safe.h"
+#include "base/time/time.h"
+#include "url/gurl.h"
+
+namespace sql {
+class Connection;
+}
+
+namespace precache {
+
+class PrecacheURLTable;
+class PrecacheStatisticsTable;
+
+// Class that tracks information related to precaching. This class can be
+// constructed or destroyed on any threads, but all other methods must be called

[bengr, 2013/10/23 19:03:36]

threads -> thread

on a single thread -> on the same thread

[sclittle, 2013/10/24 22:11:38]

Done.

+// on a single thread (e.g. the DB thread).
+class PrecacheDatabase : public base::RefCountedThreadSafe<PrecacheDatabase>,
+                         base::NonThreadSafe {
+ public:
+  // A PrecacheDatabase can be constructed on any thread.
+  PrecacheDatabase();
+
+  // Initializes the precache database with the specified connection. The
+  // PrecacheDatabase takes ownership of |db|.
+  void Init(sql::Connection* db);
+
+  // Reports UMA of precache statistics for days up to and including the day of
+  // |flush_end_date|. The precache statistics for these days are deleted from
+  // the database after they have been reported, and expired precache history is
+  // deleted from the precache URL table.
+  void FlushOldStats(base::Time flush_end_date);

[bengr, 2013/10/23 19:03:36]

ReportPreviousDaysStats() ?

[sclittle, 2013/10/24 22:11:38]

Changed to ReportAndDeleteOldStats.

Passing in a Time parameter makes testing nicer since we can use a constant, as
opposed to relying on "Now()" in tests.

+
+  // Update metrics in response to a URL being fetched.
+  void RecordURLFetched(GURL url, base::Time fetch_time, int64 size,
+                        bool was_cached, bool is_precaching, bool is_cellular);
+
+ private:
+  friend class base::RefCountedThreadSafe<PrecacheDatabase>;
+  friend class PrecacheDatabaseTest;
+  ~PrecacheDatabase();
+
+  void CreateTableIfNonExistent();
+
+  scoped_ptr<sql::Connection> db_;
+  scoped_ptr<PrecacheURLTable> precache_url_table_;
+  scoped_ptr<PrecacheStatisticsTable> precache_statistics_table_;
+
+  DISALLOW_COPY_AND_ASSIGN(PrecacheDatabase);
+};
+
+}  // namespace precache
+
+#endif  // COMPONENTS_PRECACHE_CORE_PRECACHE_DATABASE_H_