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

Unified Diff: chrome/browser/expected_install_modules_win.cc

Issue 23513049: Implement install module verification metric. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/browser/expected_install_modules_win.cc
diff --git a/chrome/browser/expected_install_modules_win.cc b/chrome/browser/expected_install_modules_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..96f59f6ad598f875f82fb96df9bfb99dc5f22e75
--- /dev/null
+++ b/chrome/browser/expected_install_modules_win.cc
@@ -0,0 +1,31 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/expected_install_modules_win.h"
+
+std::string CalculateInstalledModuleDigest(const std::string& module_name) {
+ return module_name;
+}
robertshield 2013/09/13 17:26:05 minor nit, move kExpectedInstallModuleDigests up
erikwright (departed) 2013/09/13 20:19:56 Done.
+
+// Do not reorder these elements. Their index is used as an ID in metrics
+// reporting.
+extern const char* kExpectedInstallModuleDigests[] = {
+ "chrome.dll",
+ "icudt.dll",
+ "chrome_frame_helper.dll",
+ "d3dcompiler_46.dll",
+ "libglesv2.dll",
+ "metro_driver.dll",
+ "pdf.dll",
+ "widevinecdmadapter.dll",
+ "chrome_child.dll",
+ "d3dcompiler_43.dll",
+ "ffmpegsumo.dll",
+ "libegl.dll",
+ "libpeerconnection.dll",
+ "npchrome_frame.dll",
+ "ppgooglenaclpluginchrome.dll",
+ "xinput1_3.dll",
+ 0
+};

Powered by Google App Engine
This is Rietveld 408576698