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

Issue 10832018: [net/dns] Remove check for DNS Changer. (Closed)

Created:
8 years, 5 months ago by szym
Modified:
8 years, 5 months ago
Reviewers:
cbentzel
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org, mmenke
Visibility:
Public.

Description

[net/dns] Remove check for DNS Changer. BUG=125599 TEST=No AsyncDNS.DNSChangerDetected in chrome://histograms Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=148604

Patch Set 1 #

Patch Set 2 : sync #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -57 lines) Patch
M net/dns/dns_config_service.h View 1 chunk +0 lines, -3 lines 0 comments Download
M net/dns/dns_config_service.cc View 1 3 chunks +1 line, -54 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
szym
8 years, 5 months ago (2012-07-25 22:26:14 UTC) #1
cbentzel
LGTM
8 years, 5 months ago (2012-07-26 14:27:13 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/szym@chromium.org/10832018/1
8 years, 5 months ago (2012-07-26 14:33:53 UTC) #3
commit-bot: I haz the power
Failed to apply patch for net/dns/dns_config_service.cc: While running patch -p1 --forward --force; patching file net/dns/dns_config_service.cc ...
8 years, 5 months ago (2012-07-26 14:33:54 UTC) #4
commit-bot: I haz the power
8 years, 5 months ago (2012-07-26 17:13:09 UTC) #5

Powered by Google App Engine
This is Rietveld 408576698