Include video frame memory held by the compositor in memory reports.

Include video frame memory held by the compositor in memory reports.

This provides an estimate for the amount of memory owned by the
video frame held by the compositor after the video renderer has
suspended. In this case, there is 1 frame of video memory held
by VideoFrameCompositor while the pipeline reports zero memory
since the renderer has been destroyed.

This doesn't resolve the linked bug, but does improve the accuracy
of the reported value with high resolution video.

BUG=700776
TEST=none

Review-Url: https://codereview.chromium.org/2828503004
Cr-Commit-Position: refs/heads/master@{#465435}
Committed: https://chromium.googlesource.com/chromium/src/+/ecc7661918e467da103b9e8f1c05b83265b1e707

[DaleCurtis, 2017-04-18 21:42:59]

dalecurtis@chromium.org changed reviewers:
+ sandersd@chromium.org

[DaleCurtis, 2017-04-18 21:43:31]

Description was changed from

==========
Include video frame memory held by the compositor in memory reports.

This provides an estimate for the amount of memory owned by the
video frame held by the compositor after the video renderer has
suspended. In this case, there is 1 frame of video memory held
by VideoFrameCompositor while the pipeline reports zero memory
since the renderer has been destoryed.

BUG=700776
TEST=none
==========

to

==========
Include video frame memory held by the compositor in memory reports.

This provides an estimate for the amount of memory owned by the
video frame held by the compositor after the video renderer has
suspended. In this case, there is 1 frame of video memory held
by VideoFrameCompositor while the pipeline reports zero memory
since the renderer has been destroyed.

This doesn't resolve the linked bug, but does improve the accuracy
of the reported value with high resolution video.

BUG=700776
TEST=none
==========

[sandersd (OOO until July 31), 2017-04-18 21:47:11]

lgtm

[DaleCurtis, 2017-04-18 22:00:49]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-04-18 22:01:22]

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-19 00:31:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492552849656470, "parent_rev":
"2df988056751c340253ed0ba4406598c80aaa3c0", "commit_rev":
"ecc7661918e467da103b9e8f1c05b83265b1e707"}

[commit-bot: I haz the power, 2017-04-19 00:31:52]

Description was changed from

==========
Include video frame memory held by the compositor in memory reports.

This provides an estimate for the amount of memory owned by the
video frame held by the compositor after the video renderer has
suspended. In this case, there is 1 frame of video memory held
by VideoFrameCompositor while the pipeline reports zero memory
since the renderer has been destroyed.

This doesn't resolve the linked bug, but does improve the accuracy
of the reported value with high resolution video.

BUG=700776
TEST=none
==========

to

==========
Include video frame memory held by the compositor in memory reports.

This provides an estimate for the amount of memory owned by the
video frame held by the compositor after the video renderer has
suspended. In this case, there is 1 frame of video memory held
by VideoFrameCompositor while the pipeline reports zero memory
since the renderer has been destroyed.

This doesn't resolve the linked bug, but does improve the accuracy
of the reported value with high resolution video.

BUG=700776
TEST=none

Review-Url: https://codereview.chromium.org/2828503004
Cr-Commit-Position: refs/heads/master@{#465435}
Committed:
https://chromium.googlesource.com/chromium/src/+/ecc7661918e467da103b9e8f1c05...
==========

[commit-bot: I haz the power, 2017-04-19 00:31:53]

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