Save extension activity log to a file.

chrome/browser/extensions/activity_actions.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// User of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_ACTIVITY_ACTIONS_H_
+#define CHROME_BROWSER_EXTENSIONS_ACTIVITY_ACTIONS_H_
+
+#include <string>
+#include "base/values.h"
+
+namespace activity {
+
+// This is the interface for extension actions that are to be recorded in
+// the activity log.
+class Action {
+  public:

[mvrable, 2012/12/04 01:00:13]

There's extra indentation here: the visibility specifiers ("public:") should be
indented one space only, and the rest of the contents of the class two spaces.

[felt, 2012/12/07 19:27:45]

Done.

+    // Print an action with il8n substitutions for display.
+    virtual std::string PrettyPrintForil8n() OVERRIDE;

[mvrable, 2012/12/04 01:00:13]

OVERRIDE should be used in subclasses when overriding a method in the parent. 
It shouldn't be used in the parent class.  If there's no implementation of this
method in the parent class (it's a pure virtual method), define it like this:
  virtual std::string PrettyPrintForil8n() = 0;

[felt, 2012/12/07 19:27:45]

Done.

+
+    // Print an action as a regular string for debugging purposes.
+    virtual std::string PrettyPrintForDebug() OVERRIDE;
+
+    virtual ~Action();
+};
+}

[mvrable, 2012/12/04 01:00:13]

Leave a blank line between the two closing braces, and mark the second one as
  }  // namespace activity

[felt, 2012/12/07 19:27:45]

Done.

+
+#endif  // CHROME_BROWSER_EXTENSIONS_ACTIVITY_ACTIONS_H_

[mvrable, 2012/12/04 01:00:13]

lint in the web interface complains about no newline at the end of the
file--that would be worth checking.

[felt, 2012/12/07 19:27:45]

Done.