[Manifest] Add metrics recording.

content/renderer/manifest/manifest_parser.cc

Index: content/renderer/manifest/manifest_parser.cc
diff --git a/content/renderer/manifest/manifest_parser.cc b/content/renderer/manifest/manifest_parser.cc
index a68ca2ae72e3fb14b86582ebae006612bfd7d201..5c27fceee6750717518aafb0313ef5070f4eed60 100644
--- a/content/renderer/manifest/manifest_parser.cc
+++ b/content/renderer/manifest/manifest_parser.cc
@@ -12,6 +12,7 @@
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "content/public/common/manifest.h"
+#include "content/renderer/manifest/manifest_uma_util.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace content {
@@ -316,6 +317,8 @@ Manifest ManifestParser::Parse(const base::StringPiece& json,
   manifest.icons = ParseIcons(*dictionary, manifest_url);
   manifest.gcm_sender_id = ParseGCMSenderID(*dictionary);
 
+  ManifestUmaUtil::ParsedManifest(manifest);

[Miguel Garcia, 2014/10/02 14:26:46]

since you only parse from one place I would do this at the caller site. and
depending on wether you got a a full manifest or an empty one log a manifest
valid or manifest error code.
RIght now all you have is wether the manifest is empty but you don't log parsing
errors (since you are returning early above)

[mlamouri (slow - plz ping), 2014/10/02 15:35:06]

Done.

+
   return manifest;
 }