V8ValueConverter for the activity logger that does not invoke interceptors and

chrome/renderer/extensions/activity_log_value_converter.h

+// Copyright (c) 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_RENDERER_EXTENSIONS_ACTIVITY_LOG_VALUE_CONVERTER_H_
+#define CHROME_RENDERER_EXTENSIONS_ACTIVITY_LOG_VALUE_CONVERTER_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "content/public/renderer/v8_value_converter.h"
+#include "v8/include/v8.h"
+
+namespace base {
+class Value;
+}
+
+namespace extensions {
+
+class ActivityLogValueConverter {

[felt, 2013/07/18 17:28:20]

Can you add a comment to explain why this new class is needed? You could
probably lift some of the text from the long comment in the cc file (which I
think does a fairly good job explaining).

[pmarch, 2013/07/18 20:42:03]

Done.

+ public:
+  ActivityLogValueConverter();
+  virtual ~ActivityLogValueConverter();
+
+  base::Value* FromV8Value(v8::Handle<v8::Value> value,
+                           v8::Handle<v8::Context> context) const;
+ private:
+  scoped_ptr<content::V8ValueConverter> v8_value_converter_;
+
+  DISALLOW_COPY_AND_ASSIGN(ActivityLogValueConverter);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_RENDERER_EXTENSIONS_ACTIVITY_LOG_VALUE_CONVERTER_H_

[felt, 2013/07/18 17:28:20]

linter is complaining about lack of a new line at the end of the file

[pmarch, 2013/07/18 20:42:03]

Done.