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

Issue 754716 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
please use my google.com address
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

All mash_browser_tests timeout

Project Member Reported by jonr...@chromium.org, Aug 11 2017

Issue description

Hey rockot@

Your change fda282af5fded17df0f15b61c0907aba81609ea4 https://chromium.googlesource.com/chromium/src/+/fda282af5fded17df0f15b61c0907aba81609ea4

to how Mojo Bindings handle single-phase serialization, has lead to mash_browser_tests timing out.

The first instance of this is seen on the Mojo FYI bots: https://build.chromium.org/p/chromium.fyi/builders/Mojo%20ChromiumOS/builds/20873

When I revert your change locally they all pass. I haven't dug further into where the timeouts are occurring.
 

Comment 1 by roc...@chromium.org, Aug 11 2017

I'll look at to it. Note that I'm fine with reverting for now, but r493547
also needs to be reverted first in that case.
I can land a combined revert of the two then.
Status: WontFix (was: Assigned)
Nevermind, the accused changes seemed to have just shift the timings around to lead to a timeout always happening.

Sign in to add a comment