Trace revision and v8 version for IterationInfo

chrome/common/chrome_content_client.cc

Index: chrome/common/chrome_content_client.cc
diff --git a/chrome/common/chrome_content_client.cc b/chrome/common/chrome_content_client.cc
index f4414936e8e0e89a3495aec4073693e75ea35484..80b30bfe8333eaaa2dfa734c5a6b1bce2af33784 100644
--- a/chrome/common/chrome_content_client.cc
+++ b/chrome/common/chrome_content_client.cc
@@ -46,6 +46,7 @@
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/layout.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "v8/include/v8.h"
 #include "widevine_cdm_version.h"  // In SHARED_INTERMEDIATE_DIR.
 
 #if defined(OS_LINUX)
@@ -611,6 +612,14 @@ std::string ChromeContentClient::GetProduct() const {
   return ::GetProduct();
 }
 
+std::string ChromeContentClient::GetLastChange() const {
+  return version_info::GetLastChange();

[Primiano Tucci (use gerrit), 2016/06/21 06:34:09]

Why this plumbing here? This seems to defeat the purpose of version_info.h being
a component.

[benjhayden, 2016/06/21 23:30:36]

** Presubmit ERRORS **
You added one or more #includes that violate checkdeps rules.
  content/browser/tracing/tracing_controller_impl.cc
      Illegal include: "components/version_info/version_info.h"
    Because of no rule applying.

Should I add a rule?

[Primiano Tucci (use gerrit), 2016/06/22 08:29:10]

Yup, add it to content/browser/tracing/DEPS and then you'll need a stamp from
one of the version_info owners (See components/version_info/OWNERS)

+}
+
+std::string ChromeContentClient::GetV8Version() const {
+  return v8::V8::GetVersion();
+}
+
 std::string ChromeContentClient::GetUserAgent() const {
   return ::GetUserAgent();
 }