Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Unified Diff: content/renderer/renderer_main.cc

Issue 140763008: Add a UMA stat to track if the Browser blacklist is Set on the Renderer (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add to test Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/renderer_main.cc
diff --git a/content/renderer/renderer_main.cc b/content/renderer/renderer_main.cc
index 634684c58fe2fabf8f591cbaf00159ea557d58d0..35bb64cb725ae7d2830da2519234c022507ac43b 100644
--- a/content/renderer/renderer_main.cc
+++ b/content/renderer/renderer_main.cc
@@ -48,6 +48,10 @@
#include "third_party/WebKit/public/web/WebView.h"
#endif // OS_MACOSX
+#if defined(OS_WIN)
+#include "chrome_elf/blacklist/blacklist.h"
+#endif // OS_WIN
+
#if defined(ENABLE_WEBRTC)
#include "third_party/libjingle/overrides/init_webrtc.h"
#endif
@@ -237,6 +241,13 @@ int RendererMain(const MainFunctionParams& parameters) {
platform.RunSandboxTests(no_sandbox);
+#if defined(OS_WIN)
+ // TODO(csharp): Remove once the renderer is no longer getting
+ // patched this way.
+ if (blacklist::IsBlacklistInitialized())
+ UMA_HISTOGRAM_BOOLEAN("Blacklist.PatchedInRenderer", true);
Alexei Svitkine (slow) 2014/02/06 22:12:29 I would actually suggest: UMA_HISTOGRAM_BOOLEAN("
csharp 2014/02/06 22:21:59 We just want the raw numbers since ideally this sh
+#endif
+
startup_timer.Stop(); // End of Startup Time Measurement.
if (run_loop) {

Powered by Google App Engine
This is Rietveld 408576698