New issue
Advanced search Search tips

Issue 772280 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug



Sign in to add a comment

[Missing Test]: Missing WaitSyncToken call in Canvas2DLayerBridge

Project Member Reported by krajshree@chromium.org, Oct 6 2017

Issue description

Automated tests for the below commit have been missing.Please add test coverage ASAP to avoid regressions in future.

CL: 
----
https://chromium.googlesource.com/chromium/src.git/+/e973fe29902695113010b2ec127260f0d3b08b15

Ref Bug: 
---------
https://bugs.chromium.org/p/chromium/issues/detail?id=769155

Thank you...!!
 
Labels: -ReleaseBlock-Stable
Justin, will it be possible to add a test for the above bug?

Comment 2 by junov@chromium.org, Oct 12 2017

End to end testing is not desirable because failures would be flaky and rare.  Unit testing is possible though.  I am currently in the process of refactoring this code.  Will add tests on the refactored code..

Comment 3 by junov@chromium.org, Nov 14 2017

Labels: -Pri-1 Pri-2
Owner: ----
Status: Available (was: Assigned)
Labels: -Type-Bug-Regression Type-Bug

Sign in to add a comment