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

Unified Diff: chrome/renderer/chrome_content_renderer_client.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: Update isolate files 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: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 9318da99078dc025e91563d1113aade1a511d85b..35abf402617721012f6f76aa2ce320ffa2eba6d2 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -116,6 +116,10 @@
#include "chrome/renderer/spellchecker/spellcheck_provider.h"
#endif
+#if defined(OS_WIN)
+#include "chrome_elf/blacklist/blacklist.h"
+#endif // OS_WIN
+
using autofill::AutofillAgent;
using autofill::PasswordAutofillAgent;
using autofill::PasswordGenerationAgent;
@@ -353,6 +357,19 @@ void ChromeContentRendererClient::RenderThreadStarted() {
extensions::ExtensionsClient::Set(
extensions::ChromeExtensionsClient::GetInstance());
+
+#if defined(OS_WIN)
+ // Report if the renderer process has been patched by chrome_elf.
+ // TODO(csharp): Remove once the renderer is no longer getting
+ // patched this way.
+ typedef bool(*IsBlacklistInitializedFunc)();
+ IsBlacklistInitializedFunc is_blacklist_initialized =
+ reinterpret_cast<IsBlacklistInitializedFunc>(
+ GetProcAddress(GetModuleHandle(L"chrome_elf.dll"),
+ "IsBlacklistInitialized"));
+ if (is_blacklist_initialized())
+ UMA_HISTOGRAM_BOOLEAN("Blacklist.PatchedInRenderer", true);
+#endif
}
void ChromeContentRendererClient::RenderFrameCreated(

Powered by Google App Engine
This is Rietveld 408576698