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

Issue 793296 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: ----



Sign in to add a comment

NetworkServiceRestartBrowserTest.StoragePartitionImplGetNetworkContext is Flaky

Project Member Reported by Findit, Dec 8 2017

Issue description

Labels: OS-Mac
Owner: chongz@chromium.org
Status: Started (was: Available)
Going to disable the test on Mac and investigate.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 8 2017

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

commit 8185d184ea70280bacb533526506c036176fcdfe
Author: Chong Zhang <chongz@chromium.org>
Date: Fri Dec 08 20:11:37 2017

Disable flaky NetworkServiceRestartBrowserTest.StoragePartitionImplGetNetworkContext on Mac

TBR=fdoray@chromium.org,mmenke@chromium.org

Bug:  793296 
Change-Id: I822d329925a2d6ca2af0ddcedfe6e45ba0628221
Reviewed-on: https://chromium-review.googlesource.com/817980
Reviewed-by: Chong Zhang <chongz@chromium.org>
Commit-Queue: Chong Zhang <chongz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522855}
[modify] https://crrev.com/8185d184ea70280bacb533526506c036176fcdfe/content/browser/network_service_restart_browsertest.cc

Project Member

Comment 3 by bugdroid1@chromium.org, Dec 9 2017

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

commit e0dfdb630ff40fb25ba2a117c7ec07d425de7350
Author: Chong Zhang <chongz@chromium.org>
Date: Sat Dec 09 18:25:14 2017

Flush network_service_instance after simulating crash

Make sure 'network_service_instance.cc::g_network_service' receives
error notification by flushing its InterfacePtr.

Related tests were potentially flaky but only observed on Mac bot.

Bug:  793296 
Change-Id: Ia2e3a3bcc0b2bed28b86d2b119c0fd42df939112
Reviewed-on: https://chromium-review.googlesource.com/817642
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Chong Zhang <chongz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523006}
[modify] https://crrev.com/e0dfdb630ff40fb25ba2a117c7ec07d425de7350/chrome/browser/chrome_network_service_restart_browsertest.cc
[modify] https://crrev.com/e0dfdb630ff40fb25ba2a117c7ec07d425de7350/content/browser/network_service_restart_browsertest.cc
[modify] https://crrev.com/e0dfdb630ff40fb25ba2a117c7ec07d425de7350/content/public/browser/network_service_instance.cc
[modify] https://crrev.com/e0dfdb630ff40fb25ba2a117c7ec07d425de7350/content/public/browser/network_service_instance.h

Status: Fixed (was: Started)

Sign in to add a comment