V8ValueConverter for the activity logger that does not invoke interceptors and

chrome/renderer/extensions/activity_log_converter_strategy.cc

+// 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.
+
+#include "chrome/renderer/extensions/activity_log_converter_strategy.h"
+#include "base/values.h"
+
+namespace extensions {
+
+bool ActivityLogConverterStrategy::FromV8Object(v8::Handle<v8::Object> value,
+                                                base::Value** out) const {
+  return FromV8ObjectInternal(value, out);
+}
+
+bool ActivityLogConverterStrategy::FromV8Array(v8::Handle<v8::Array> value,
+                                               base::Value** out) const {
+  return FromV8ObjectInternal(value, out);
+}
+
+bool ActivityLogConverterStrategy::FromV8ObjectInternal(
+    v8::Handle<v8::Object> value,
+    base::Value** out) const {
+
+  // Handle JSObject.
+  // We cannot use value->Get(key/index) as there may be a getter method,
+  // accessor, interceptor/handler, or access check callback set on the
+  // property. If that it is the case, any of those may invoke JS code that
+  // may result on logging extension activity events caused by value conversion
+  // rather than extension itself.
+

[not at google - send to devlin, 2013/08/07 22:07:56]

add a comment about the array index stuff we've been talking about.

[pmarch, 2013/08/08 00:53:07]

Done.

+  v8::Handle<v8::String> name = v8::String::New("[");
+  if (value->IsFunction()) {
+    name =  v8::String::Concat(name, v8::String::New("Function"));
+    v8::Handle<v8::Value> fname =
+        v8::Handle<v8::Function>::Cast(value)->GetName();
+    if (fname->IsString() && v8::Handle<v8::String>::Cast(fname)->Length()) {
+      name = v8::String::Concat(name, v8::String::New(" "));
+      name = v8::String::Concat(name, v8::Handle<v8::String>::Cast(fname));
+      name = v8::String::Concat(name, v8::String::New("()"));
+    }
+  } else {
+    name = v8::String::Concat(name, value->ToObject()->GetConstructorName());
+  }
+  name = v8::String::Concat(name, v8::String::New("]"));
+  v8::String::Utf8Value utf8(name);
+  *out = new base::StringValue(std::string(*utf8, utf8.length()));

[not at google - send to devlin, 2013/08/07 22:07:56]

you should be able to do just

name = v8::String::Concat(...);
*out = new base::StringValue(*v8::String::AsciiValue(name))

[not at google - send to devlin, 2013/08/07 22:08:40]

where by AsciiValue I mean Utf8Value. Or is the length() part really
necessary..?

[pmarch, 2013/08/08 00:53:07]

Done.

[pmarch, 2013/08/08 00:53:07]

Done.

+  return true; // Prevent V8ValueConverter from further processing this object.

[not at google - send to devlin, 2013/08/07 22:07:56]

comment would look better on its own line

[pmarch, 2013/08/08 00:53:07]

Done.

+}
+
+}  // namespace extensions