Rationalize and fix chromium android linker histogram recording.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index b931da5d1cdfb511540de0ed3f27793a1968ecaf..7a803225c791abac05a09cad3179971b2511db89 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -198,6 +198,7 @@
 #if defined(OS_ANDROID)
 #include <cpu-features.h>
 
+#include "base/android/library_loader/library_loader_hooks.h"
 #include "content/renderer/android/address_detector.h"
 #include "content/renderer/android/content_detector.h"
 #include "content/renderer/android/email_detector.h"
@@ -823,6 +824,11 @@ void RenderViewImpl::Initialize(RenderViewImplParams* params) {
 
   ProcessViewLayoutFlags(command_line);
 
+#if defined(OS_ANDROID)
+  // If we have a pending chromium android linker histogram, record it.
+  base::android::RecordChromiumAndroidLinkerRendererHistogram();

[jamesr, 2014/10/02 20:26:48]

why is this call here? this function is called many times over the life of a
render process

[simonb (inactive), 2014/10/03 11:51:42]

It needs to run after stats/histogram are started, otherwise the histogram
record is simply dropped.  If you can suggest a better location I'm happy to
move it.  The only requirement is that histograms and IPC should be available at
that point.

+#endif
+
   GetContentClient()->renderer()->RenderViewCreated(this);
 
   // If we have an opener_id but we weren't created by a renderer, then