Issue metadata
Sign in to add a comment
|
NetworkService: Some DownloadContentTest are flaky due to wierd -21 error code |
||||||||||||||||||
Issue descriptionI ran DownloadContentTest.ResumeRestoredDownload_NoFile on my linux box today, and found it failed sometimes when DownloadResponseHandler::OnComplete() was called with a -21 network error code, before OnReceiveResponse() was ever called. -21 is NETWORK_CHANGED. Any one has any idea about this? There doesn't seems to be any other code after creating the URLLoader and URLLoaderClient in resource_downloader.cc FYI, mojo.fyi linux bot doesn't fail on this test.
,
Nov 15 2017
Sorry, that should be "we are using one in browsertests with the network service enabled."
,
Nov 15 2017
The only difference between these flaky tests and the other tests is that they are creating a ranged request (ResumeRestoredDownload_xxxx), rather than starting from offset 0. if a test starts from 0, it seems fine
,
Nov 16 2017
@qinmin: I'm surprised you're seeing ERR_NETWORK_CHANGED. Was your network changing? e.g. were you plugging/unplugging Ethernet or leaving/joining WiFi networks? Can you run "ip monitor" from the command line to see what changes appear. If you're using a linux desktop there should be almost no network changes on a regular basis. @mmenke: There are a few ways to disable the real NCN: net::test::ScopedMockNetworkChangeNotifier net::test::MockNetworkChangeNotifier net::NetworkChangeNotifier::DisableForTest net::NetworkChangeNotifier::CreateMock Various individual tests use one of these to disable the NCN. I think only net_unittests disables the real NCN for the whole test suite. Generally our bots shouldn't encounter network changes (the Android bots don't have any active network AFAIK). I'd like to figure out why qinmin@ experienced a network change before working on disabling the NCN for his tests.
,
Feb 16 2018
,
Jan 10
Downgrading P2s that haven't been modified in more than 6 months, which have no component or owner. |
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by mmenke@chromium.org
, Nov 15 2017