Compressed activity log database storage

chrome/browser/extensions/activity_log/counting_policy.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 CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_COUNTING_POLICY_H_
+#define CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_COUNTING_POLICY_H_
+
+#include <string>
+
+#include "base/containers/hash_tables.h"
+#include "chrome/browser/extensions/activity_log/stream_noargs_ui_policy.h"
+
+namespace extensions {
+
+// A policy for logging the stream of actions, but without arguments.
+class CountingPolicy : public StreamWithoutArgsUIPolicy {

[felt, 2013/08/02 16:36:28]

Is there a reason to keep the StreamWithoutArgsUIPolicy around and separate? The
only historical reason that exists is because I once did a halfassed privacy
hack, which was to put a flag around recording arguments. ;) However, if you
think it makes sense to keep it now, that's fine.

[mvrable, 2013/08/02 17:16:58]

Probably not.  I need to do some cleanup work here--right now I think URL
sanitization isn't being done in the new policy.  I might pull out common code
and perhaps get rid of StreamWithoutArgsUIPolicy.

+ public:
+  explicit CountingPolicy(Profile* profile);
+  virtual ~CountingPolicy();
+
+  /*virtual void ProcessAction(scoped_refptr<Action> action) OVERRIDE;*/
+
+  static const char* kTableName;
+  static const char* kTableContentFields[];
+  static const char* kTableFieldTypes[];
+
+ protected:
+  // Implementing ActivityDatabase::Delegate.
+  virtual bool InitDatabase(sql::Connection* db) OVERRIDE;
+  virtual bool FlushDatabase(sql::Connection* db) OVERRIDE;
+
+ private:
+  /*void QueueAction(scoped_refptr<Action> action);*/
+
+  /*base::hash_set<std::string> arg_whitelist_api_;*/
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_COUNTING_POLICY_H_