When the system proc based DNS resolver returns 127.0.53.53, we fail with ERR_ICANN_NAME_COLLISION. This is implemented by HostResolverImpl. However this is not working when you use the async/builtin DNS resolver.
Nice catch!
(I sourced this from https://bugs.chromium.org/p/chromium/issues/detail?id=479620#c52, where this issue can be seen in a log)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1efc237c5f0ef90195c3dd54e2d63f3e056194be commit 1efc237c5f0ef90195c3dd54e2d63f3e056194be Author: eroman <eroman@chromium.org> Date: Wed Dec 14 00:00:45 2016 Make ERR_ICANN_NAME_COLLISION work for async DNS resolver. BUG= 670093 Review-Url: https://codereview.chromium.org/2567623003 Cr-Commit-Position: refs/heads/master@{#438346} [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/chrome/browser/net/dns_probe_browsertest.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/chrome/browser/net/dns_probe_runner_unittest.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/chrome/browser/net/dns_probe_service_unittest.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/chrome/browser/net/dns_probe_test_util.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/chrome/browser/net/dns_probe_test_util.h [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/net/dns/dns_test_util.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/net/dns/dns_test_util.h [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/net/dns/host_resolver_impl.cc [modify] https://crrev.com/1efc237c5f0ef90195c3dd54e2d63f3e056194be/net/dns/host_resolver_impl_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0cd87b66f47aa7f08c09eab08f9c6303dd107766 commit 0cd87b66f47aa7f08c09eab08f9c6303dd107766 Author: eroman <eroman@chromium.org> Date: Wed Dec 14 19:13:45 2016 Misc HostResolverImpl refactors. * Don't cache ERR_ICANN_NAME_COLLISION under task-based resolver (to match proc-based resolver) * Save an empty address list in cache for ERR_ICANN_NAME_COLLISION entries (rather than the address list originally resolved). BUG= 670093 Review-Url: https://codereview.chromium.org/2577503003 Cr-Commit-Position: refs/heads/master@{#438575} [modify] https://crrev.com/0cd87b66f47aa7f08c09eab08f9c6303dd107766/net/dns/host_resolver_impl.cc [modify] https://crrev.com/0cd87b66f47aa7f08c09eab08f9c6303dd107766/net/dns/host_resolver_impl_unittest.cc
Comment 1 by mmenke@chromium.org
, Dec 1 2016