• Fady Samuel's avatar
    Revert "Surface synchronization: Don't report blocked duration of throttled children" · cc0e9d5e
    Fady Samuel authored
    This reverts commit d15ba9ba.
    
    Reason for revert: This doesn't seem to solve the deadline reporting bug. This was a speculative fix that didn't fix the issue.
    
    Original change's description:
    > Surface synchronization: Don't report blocked duration of throttled children
    > 
    > A child can be "throttled" for an arbitrary length of time if it is never
    > embedded. This skews blocked duration metrics. Simply don't report those
    > metrics if a surface is blocked on a parent.
    > 
    > Bug: 890767
    > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
    > Change-Id: I3c8a8494a5b1e5a973b2e5c2b61cffe5c2b76f6f
    > Reviewed-on: https://chromium-review.googlesource.com/c/1274591
    > Commit-Queue: Fady Samuel <fsamuel@chromium.org>
    > Reviewed-by: Saman Sami <samans@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#598886}
    
    TBR=fsamuel@chromium.org,samans@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 890767
    Change-Id: I3b1188d49a1301681391d526abaa25dcabf69f79
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
    Reviewed-on: https://chromium-review.googlesource.com/c/1281083Reviewed-by: default avatarFady Samuel <fsamuel@chromium.org>
    Commit-Queue: Fady Samuel <fsamuel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599716}
    cc0e9d5e
surface.cc 26.7 KB