Revert of Discard compositor frames from unloaded web content

Revert of Discard compositor frames from unloaded web content (patchset #11 id:200001 of https://codereview.chromium.org/2707243005/ )

Reason for revert:
It looks like this broke the MSAN bot:

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN/builds/731

Original issue's description:
> Discard compositor frames from unloaded web content
> 
> After a navigation commits, it is still possible for the compositor to
> resend a frame that draws the previous (now unloaded) page. In some
> situations this works against the browser process' new content
> rendering timer that attempts to clear a page after 4 seconds if a
> navigation has committed but not yet painted in that time.
> 
> This CL adds a counter called content_source_id to that increments on
> every non-same-page navigation commit under a given top-level
> RenderWidget. It tags the CompositorFrameMetadata with this identifier,
> and also sends it to the browser process with the
> DidCommitProvisionaLoad IPC message. This enables the browser to
> recognize compositor frames with stale IDs, corresponding to unloaded
> pages.
> 
> BUG=672847
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2707243005
> Cr-Commit-Position: refs/heads/master@{#454578}
> Committed: https://chromium.googlesource.com/chromium/src/+/efd1189a0b3febada2bceb4c037fff1718306a3a

TBR=enne@chromium.org,creis@chromium.org,danakj@chromium.org,kenrb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672847
Review-Url: https://codereview.chromium.org/2730203002
Cr-Commit-Position: refs/heads/master@{#454732}
Committed: https://chromium.googlesource.com/chromium/src/+/cee456faba216e014f2ce2ded37e2cde5fc19cfd

[sclittle, 2017-03-04 00:54:49]

The CQ bit was checked by sclittle@chromium.org

[sclittle, 2017-03-04 00:54:49]

Created Revert of Discard compositor frames from unloaded web content

[commit-bot: I haz the power, 2017-03-04 00:55: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-03-04 00:57:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488588889069280, "parent_rev":
"08e64091e1aff0f7efe873761fab7ace1a307fc5", "commit_rev":
"cee456faba216e014f2ce2ded37e2cde5fc19cfd"}

[commit-bot: I haz the power, 2017-03-04 00:57:49]

Description was changed from

==========
Revert of Discard compositor frames from unloaded web content (patchset #11
id:200001 of https://codereview.chromium.org/2707243005/ )

Reason for revert:
It looks like this broke the MSAN bot:

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Original issue's description:
> Discard compositor frames from unloaded web content
> 
> After a navigation commits, it is still possible for the compositor to
> resend a frame that draws the previous (now unloaded) page. In some
> situations this works against the browser process' new content
> rendering timer that attempts to clear a page after 4 seconds if a
> navigation has committed but not yet painted in that time.
> 
> This CL adds a counter called content_source_id to that increments on
> every non-same-page navigation commit under a given top-level
> RenderWidget. It tags the CompositorFrameMetadata with this identifier,
> and also sends it to the browser process with the
> DidCommitProvisionaLoad IPC message. This enables the browser to
> recognize compositor frames with stale IDs, corresponding to unloaded
> pages.
> 
> BUG=672847
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2707243005
> Cr-Commit-Position: refs/heads/master@{#454578}
> Committed:
https://chromium.googlesource.com/chromium/src/+/efd1189a0b3febada2bceb4c037f...

TBR=enne@chromium.org,creis@chromium.org,danakj@chromium.org,kenrb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672847
==========

to

==========
Revert of Discard compositor frames from unloaded web content (patchset #11
id:200001 of https://codereview.chromium.org/2707243005/ )

Reason for revert:
It looks like this broke the MSAN bot:

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Original issue's description:
> Discard compositor frames from unloaded web content
> 
> After a navigation commits, it is still possible for the compositor to
> resend a frame that draws the previous (now unloaded) page. In some
> situations this works against the browser process' new content
> rendering timer that attempts to clear a page after 4 seconds if a
> navigation has committed but not yet painted in that time.
> 
> This CL adds a counter called content_source_id to that increments on
> every non-same-page navigation commit under a given top-level
> RenderWidget. It tags the CompositorFrameMetadata with this identifier,
> and also sends it to the browser process with the
> DidCommitProvisionaLoad IPC message. This enables the browser to
> recognize compositor frames with stale IDs, corresponding to unloaded
> pages.
> 
> BUG=672847
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2707243005
> Cr-Commit-Position: refs/heads/master@{#454578}
> Committed:
https://chromium.googlesource.com/chromium/src/+/efd1189a0b3febada2bceb4c037f...

TBR=enne@chromium.org,creis@chromium.org,danakj@chromium.org,kenrb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672847

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

[commit-bot: I haz the power, 2017-03-04 00:57:51]

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