[Merge M54] aw: Fix effective frame source leak after detach

[Merge M54] aw: Fix effective frame source leak after detach

shouqun@ discovered that destroying the SurfaceFactory before all
resources belonging to that surface are returned will lead to those
resources being dropped completely, since cc::Display only holds onto a
weakptr to SurfaceFactory to return resources.

Webview goes through this cycle each time it's detached and re-attached,
so the "lost" resources will keep growing until the memory budget is
reached. Note though that destroying the webview does free these
resources, so they are not lost forever. But detach/attach is very
common, and this is still a severe leak.

Fix by submitting an empty frame into the Surface, which forces
resources from the previous frame to be returned.

BUG=655434
Review-Url: https://codereview.chromium.org/2423473002
Cr-Commit-Position: refs/heads/master@{#425466}
(cherry picked from commit 2fd5e46f7fba4945e86a5c4cd9297a6eaf2417df)

Committed: https://chromium.googlesource.com/chromium/src/+/56a61bc20d6a87ac2526ab8890b25a8d34973579

[boliu, 2016-10-14 23:52:43]

Description was changed from

==========
[Merge M54] aw: Fix effective frame source leak after detach

shouqun@ discovered that destroying the SurfaceFactory before all
resources belonging to that surface are returned will lead to those
resources being dropped completely, since cc::Display only holds onto a
weakptr to SurfaceFactory to return resources.

Webview goes through this cycle each time it's detached and re-attached,
so the "lost" resources will keep growing until the memory budget is
reached. Note though that destroying the webview does free these
resources, so they are not lost forever. But detach/attach is very
common, and this is still a severe leak.

Fix by submitting an empty frame into the Surface, which forces
resources from the previous frame to be returned.

BUG=655434

Review-Url: https://codereview.chromium.org/2423473002
Cr-Commit-Position: refs/heads/master@{#425466}
(cherry picked from commit 2fd5e46f7fba4945e86a5c4cd9297a6eaf2417df)
==========

to

==========
[Merge M54] aw: Fix effective frame source leak after detach

shouqun@ discovered that destroying the SurfaceFactory before all
resources belonging to that surface are returned will lead to those
resources being dropped completely, since cc::Display only holds onto a
weakptr to SurfaceFactory to return resources.

Webview goes through this cycle each time it's detached and re-attached,
so the "lost" resources will keep growing until the memory budget is
reached. Note though that destroying the webview does free these
resources, so they are not lost forever. But detach/attach is very
common, and this is still a severe leak.

Fix by submitting an empty frame into the Surface, which forces
resources from the previous frame to be returned.

BUG=655434

Review-Url: https://codereview.chromium.org/2423473002
Cr-Commit-Position: refs/heads/master@{#425466}
(cherry picked from commit 2fd5e46f7fba4945e86a5c4cd9297a6eaf2417df)

Committed:
https://chromium.googlesource.com/chromium/src/+/56a61bc20d6a87ac2526ab8890b2...
==========

[boliu, 2016-10-14 23:52:45]

Committed patchset #1 (id:1) manually as
56a61bc20d6a87ac2526ab8890b25a8d34973579.