ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

In the case where DelegatedFrameHost has no frame, it is not showing
a renderer tab contents, so it does not need to try prevent guttering.
However it would grab a resize lock still, in some cases where it will
never be receiving a frame from the renderer. When that happens the
lock times out after 67ms and it grabs it again.. forever. This causes
the UI to update at 15 fps instead of 60.

This fixes it by not allowing a resize lock to be grabbed when
|has_frame_| is false. And when |has_frame_| changes from true to
false, any current resize lock is dropped, freeing up the UI to
present itself.

NOTRY=true
NOPRESUBMIT=true
TBR=danakj
BUG=712903
Review-Url: https://codereview.chromium.org/2832043002
Cr-Commit-Position: refs/heads/master@{#466508}
(cherry picked from commit 8230eff799f34889d9ec93944a4e8f3147b91c17)

Review-Url: https://codereview.chromium.org/2840063002
Cr-Commit-Position: refs/branch-heads/3071@{#205}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/344425197d426ed2c3f3b5527a67f98fb0d483b5

[danakj, 2017-04-25 20:57:43]

LGTM

[danakj, 2017-04-25 20:59:54]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2017-04-25 21:01:01]

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-04-25 21:01:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-25 21:01:05]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[danakj, 2017-04-25 21:03:17]

Description was changed from

==========
ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

In the case where DelegatedFrameHost has no frame, it is not showing
a renderer tab contents, so it does not need to try prevent guttering.
However it would grab a resize lock still, in some cases where it will
never be receiving a frame from the renderer. When that happens the
lock times out after 67ms and it grabs it again.. forever. This causes
the UI to update at 15 fps instead of 60.

This fixes it by not allowing a resize lock to be grabbed when
|has_frame_| is false. And when |has_frame_| changes from true to
false, any current resize lock is dropped, freeing up the UI to
present itself.

TBR=danakj
BUG=712903

Review-Url: https://codereview.chromium.org/2832043002
Cr-Commit-Position: refs/heads/master@{#466508}
(cherry picked from commit 8230eff799f34889d9ec93944a4e8f3147b91c17)
==========

to

==========
ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

In the case where DelegatedFrameHost has no frame, it is not showing
a renderer tab contents, so it does not need to try prevent guttering.
However it would grab a resize lock still, in some cases where it will
never be receiving a frame from the renderer. When that happens the
lock times out after 67ms and it grabs it again.. forever. This causes
the UI to update at 15 fps instead of 60.

This fixes it by not allowing a resize lock to be grabbed when
|has_frame_| is false. And when |has_frame_| changes from true to
false, any current resize lock is dropped, freeing up the UI to
present itself.

NOTRY=true
NOPRESUBMIT=true
TBR=danakj
BUG=712903

Review-Url: https://codereview.chromium.org/2832043002
Cr-Commit-Position: refs/heads/master@{#466508}
(cherry picked from commit 8230eff799f34889d9ec93944a4e8f3147b91c17)
==========

[danakj, 2017-04-25 21:03:20]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2017-04-25 21:04:32]

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-04-25 21:07:59]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493154200720130,
"parent_rev": "cd81163b1811d467baf68d0316cb163d22e320c1", "commit_rev":
"344425197d426ed2c3f3b5527a67f98fb0d483b5"}

[commit-bot: I haz the power, 2017-04-25 21:08:10]

Description was changed from

==========
ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

In the case where DelegatedFrameHost has no frame, it is not showing
a renderer tab contents, so it does not need to try prevent guttering.
However it would grab a resize lock still, in some cases where it will
never be receiving a frame from the renderer. When that happens the
lock times out after 67ms and it grabs it again.. forever. This causes
the UI to update at 15 fps instead of 60.

This fixes it by not allowing a resize lock to be grabbed when
|has_frame_| is false. And when |has_frame_| changes from true to
false, any current resize lock is dropped, freeing up the UI to
present itself.

NOTRY=true
NOPRESUBMIT=true
TBR=danakj
BUG=712903

Review-Url: https://codereview.chromium.org/2832043002
Cr-Commit-Position: refs/heads/master@{#466508}
(cherry picked from commit 8230eff799f34889d9ec93944a4e8f3147b91c17)
==========

to

==========
ui: Prevent DelegatedFrameHost from taking CompositorResizeLock forever

In the case where DelegatedFrameHost has no frame, it is not showing
a renderer tab contents, so it does not need to try prevent guttering.
However it would grab a resize lock still, in some cases where it will
never be receiving a frame from the renderer. When that happens the
lock times out after 67ms and it grabs it again.. forever. This causes
the UI to update at 15 fps instead of 60.

This fixes it by not allowing a resize lock to be grabbed when
|has_frame_| is false. And when |has_frame_| changes from true to
false, any current resize lock is dropped, freeing up the UI to
present itself.

NOTRY=true
NOPRESUBMIT=true
TBR=danakj
BUG=712903

Review-Url: https://codereview.chromium.org/2832043002
Cr-Commit-Position: refs/heads/master@{#466508}
(cherry picked from commit 8230eff799f34889d9ec93944a4e8f3147b91c17)

Review-Url: https://codereview.chromium.org/2840063002
Cr-Commit-Position: refs/branch-heads/3071@{#205}
Cr-Branched-From:
a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed:
https://chromium.googlesource.com/chromium/src/+/344425197d426ed2c3f3b5527a67...
==========

[commit-bot: I haz the power, 2017-04-25 21:08:17]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/344425197d426ed2c3f3b5527a67...