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

Issue 2132693002: Merge M53: Add support for obtaining AV products on Win7. (Closed)

Created:
4 years, 5 months ago by Will Harris
Modified:
4 years, 5 months ago
Reviewers:
CC:
chromium-reviews, asvitkine+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2785
Target Ref:
refs/pending/branch-heads/2785
Project:
chromium
Visibility:
Public.

Description

Merge M53: Add support for obtaining AV products on Win7. Previously WSC API was used which is only available on Windows 8. This CL adds support via WMI, which is available from Vista onwards. BUG=615154 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng Committed: https://crrev.com/1e8be206b2e8a96af660720bef1450f8a06af167 Review-Url: https://codereview.chromium.org/2064313004 Cr-Original-Commit-Position: refs/heads/master@{#400496} Cr-Commit-Position: refs/heads/master@{#403544} (cherry picked from commit 6813e4fdd732d11c532c02fd07e16f6c76df78ab) Committed: https://chromium.googlesource.com/chromium/src/+/b75c31bb3974d9749085aad6f2420cf7e9d731af

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+205 lines, -5 lines) Patch
M chrome/browser/metrics/antivirus_metrics_provider_win.h View 2 chunks +16 lines, -2 lines 0 comments Download
M chrome/browser/metrics/antivirus_metrics_provider_win.cc View 7 chunks +174 lines, -3 lines 0 comments Download
M chrome/browser/metrics/antivirus_metrics_provider_win_unittest.cc View 3 chunks +10 lines, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Will Harris
4 years, 5 months ago (2016-07-07 17:03:23 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
b75c31bb3974d9749085aad6f2420cf7e9d731af.

Powered by Google App Engine
This is Rietveld 408576698