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

Issue 620866 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
please use my google.com address
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug



Sign in to add a comment

IPCSyncChannelTest.ChattyServer failing on Win7 (dbg)

Project Member Reported by mfomitchev@chromium.org, Jun 16 2016

Issue description

Cc: grunell@chromium.org dalecur...@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 16 2016

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

commit 7b7e60fe742530a34e7480307ae4d58d2f61eafa
Author: rockot <rockot@chromium.org>
Date: Thu Jun 16 23:14:40 2016

Revert IPC SyncChannel to base::WaitableEvent waiting

The new Mojo sync waiting behavior is causing some shutdown
hangs on Mac and test flake on Windows. Reverting for now
until that's sorted out.

Note that this behavior was landed as part of a larger change
which does not need to be reverted entirely, hence this
partial revert.

BUG= 620866 ,620683
TBR=jam@chromium.org since it's just a revert

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

[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_channel.cc
[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_channel.h
[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_message.cc
[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_message.h
[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_message_filter.cc
[modify] https://crrev.com/7b7e60fe742530a34e7480307ae4d58d2f61eafa/ipc/ipc_sync_message_filter.h

Comment 3 by roc...@chromium.org, Jun 16 2016

Status: Fixed (was: Assigned)

Sign in to add a comment