Evicted frames should immediately unblock dependent CompositorFrames

Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately 
unblock its dependent Surfaces. This causes parent Frames to block 
until the deadline hits. This can cause unnecessary jank during cross-
process navigation and may cause the browser UI to become unresponsive 
briefly. This CL immediately unblocks the parent CompositorFrame if the 
child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2689213003
Cr-Commit-Position: refs/heads/master@{#450165}
Committed: https://chromium.googlesource.com/chromium/src/+/2d9cf213310be917ade1a3f21d665fc0511e1e10

[Fady Samuel, 2017-02-13 21:06:18]

Description was changed from

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately unblock
its dependent Surfaces. This causes parent Frames to block until the deadline
hits. This can cause unnecessary jank during cross-process navigation and may
cause the browser UI to become unresponsive briefly. This CL immediately
unblocks the parent CompositorFrame if the child Surface is evicted.

BUG=689725
==========

to

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately unblock
its dependent Surfaces. This causes parent Frames to block until the deadline
hits. This can cause unnecessary jank during cross-process navigation and may
cause the browser UI to become unresponsive briefly. This CL immediately
unblocks the parent CompositorFrame if the child Surface is evicted.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Fady Samuel, 2017-02-13 21:07:19]

fsamuel@chromium.org changed reviewers:
+ kylechar@chromium.org, vmpstr@chromium.org

[Fady Samuel, 2017-02-13 21:07:20]

+vmpstr@ for review
+kylechar@ FYI

[Fady Samuel, 2017-02-13 21:13:31]

Description was changed from

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately unblock
its dependent Surfaces. This causes parent Frames to block until the deadline
hits. This can cause unnecessary jank during cross-process navigation and may
cause the browser UI to become unresponsive briefly. This CL immediately
unblocks the parent CompositorFrame if the child Surface is evicted.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately unblock
its dependent Surfaces. This causes parent Frames to block until the deadline
hits. This can cause unnecessary jank during cross-process navigation and may
cause the browser UI to become unresponsive briefly. This CL immediately
unblocks the parent CompositorFrame if the child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Fady Samuel, 2017-02-13 21:14:06]

Description was changed from

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately unblock
its dependent Surfaces. This causes parent Frames to block until the deadline
hits. This can cause unnecessary jank during cross-process navigation and may
cause the browser UI to become unresponsive briefly. This CL immediately
unblocks the parent CompositorFrame if the child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately 
unblock its dependent Surfaces. This causes parent Frames to block 
until the deadline hits. This can cause unnecessary jank during cross-
process navigation and may cause the browser UI to become unresponsive 
briefly. This CL immediately unblocks the parent CompositorFrame if the 
child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[vmpstr, 2017-02-13 22:21:46]

lgtm

[Fady Samuel, 2017-02-13 22:23:12]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2017-02-13 22:24:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-14 00:09:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487024592422100, "parent_rev":
"f68376dd95b0f67bb510769c6a4933f85637b6af", "commit_rev":
"2d9cf213310be917ade1a3f21d665fc0511e1e10"}

[commit-bot: I haz the power, 2017-02-14 00:10:11]

Description was changed from

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately 
unblock its dependent Surfaces. This causes parent Frames to block 
until the deadline hits. This can cause unnecessary jank during cross-
process navigation and may cause the browser UI to become unresponsive 
briefly. This CL immediately unblocks the parent CompositorFrame if the 
child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Evicted frames should immediately unblock dependent CompositorFrames

Prior to this CL, if a Surface is evicted, we would not immediately 
unblock its dependent Surfaces. This causes parent Frames to block 
until the deadline hits. This can cause unnecessary jank during cross-
process navigation and may cause the browser UI to become unresponsive 
briefly. This CL immediately unblocks the parent CompositorFrame if the 
child Surface is evicted.

Note that this CL is a pre-requisite to allowing SurfaceFactory to 
always observe its current Surface, which is necessary so that it is
informed when referenced surfaces change and can tell its client to
update its reference tracking. This allows the garbage collection
system to immediately garbage collect surfaces that are no longer
referenced from a parent Surface once a pending frame has replaced
an existing active frame. That change is coming in a subsequent CL.

BUG=689725
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2689213003
Cr-Commit-Position: refs/heads/master@{#450165}
Committed:
https://chromium.googlesource.com/chromium/src/+/2d9cf213310be917ade1a3f21d66...
==========

[commit-bot: I haz the power, 2017-02-14 00:10:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2d9cf213310be917ade1a3f21d66...