New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 728808 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug

Blocking:
issue 729320



Sign in to add a comment

net unit test NoIPv6OnWifi causes other tests to crash on iOS

Project Member Reported by baxley@chromium.org, Jun 1 2017

Issue description

Starting around April 13, NoIPv6OnWifi started causing HttpNetworkTransactionTest.Basic to crash.

On re-run HttpNetworkTransactionTests pass since NoIPv6OnWifi isn't run.

Here is a good build: https://build.chromium.org/p/chromium.mac/builders/ios-simulator/builds/16573

Here is a build where it crashed: https://build.chromium.org/p/chromium.mac/builders/ios-simulator/builds/16574
 
Owner: mge...@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4a9d072c5f2d8aee7e1d2439b252ddb675ed0d28

commit 4a9d072c5f2d8aee7e1d2439b252ddb675ed0d28
Author: baxley <baxley@chromium.org>
Date: Thu Jun 01 23:05:42 2017

Disable NoIPv6OnWifi on iOS.

This test causes HttpNetworkTransactionTests tests to crash on iOS.

BUG= 728808 
TBR=mef

Review-Url: https://codereview.chromium.org/2920733003
Cr-Commit-Position: refs/heads/master@{#476471}

[modify] https://crrev.com/4a9d072c5f2d8aee7e1d2439b252ddb675ed0d28/net/dns/host_resolver_impl_unittest.cc

Comment 3 by rch@chromium.org, Jun 3 2017

Blocking: 729320
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 5 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce64e673564b8a2b0912c07567e96c97aa4dca06

commit ce64e673564b8a2b0912c07567e96c97aa4dca06
Author: rch <rch@chromium.org>
Date: Mon Jun 05 17:41:57 2017

Change HostResolverImplDnsTest.NoIPv6OnWifi to no longer leak
deleted resolvers into the network change notifier.

BUG= 729320 , 728808 

Review-Url: https://codereview.chromium.org/2916903006
Cr-Commit-Position: refs/heads/master@{#477013}

[modify] https://crrev.com/ce64e673564b8a2b0912c07567e96c97aa4dca06/net/dns/host_resolver_impl_unittest.cc

Cc: mge...@chromium.org
Owner: rch@chromium.org
Status: Fixed (was: Assigned)
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment