Mac overlays: Clean up partial swap code

Mac overlays: Clean up partial swap code

Remove use of linked_ptr, because it's not necessary now that we have
C++11 support. Separate out functions to be a bit easier to read and
follow. Remove unnecessary uses of RectF when Rect would do.

BUG=533681
Review URL: https://codereview.chromium.org/1643043002

Cr-Commit-Position: refs/heads/master@{#372707}
(cherry picked from commit 4d0aa96265cd731ddd74d43821df7feb9c55c1b1)

Mac overlays: Move CALayerPartialDamageTree to its own file

Direct copy-paste, no changes.

BUG=533681
TBR=erikchen

Review URL: https://codereview.chromium.org/1647523002

Cr-Commit-Position: refs/heads/master@{#371871}
(cherry picked from commit 9a9014a549f126db8ad2dc0c0a1dc6bb246423ae)

Mac overlays: Isolate partial swap support

The logic to maintain a CALayer tree for partial swap support is
complicated and should be removed, but that would compromise battery
usage. Once the CoreAniamtion renderer is handling ~99% of the frames,
we can remove it. In the mean time, isolate it.

Create a CALayerPartialDamageTree to track the layer tree of partially
swapped backbuffers. Do this in-place to minimize diffs, so that this
patch can be reasonably reviewed. A follow-on patch will move it to
a separate file.

Slightly reorganize the logic so that CALayerPartialDamageTree matches
the general structure of CALayerTree, where this a pending (not swapped)
tree, a tree for each pending swap, and a tree for what is currently
being displayed.

BUG=533681
Review URL: https://codereview.chromium.org/1640093002

Cr-Commit-Position: refs/heads/master@{#371831}
(cherry picked from commit 8aa00fc2a374fca3e9b9f34d838db7bb5f4fc088)

Committed: https://chromium.googlesource.com/chromium/src/+/04f51c60da4daf1eb228d46954e4a8f9d23fbea1

[ccameron, 2016-02-02 20:35:40]

Description was changed from

==========
Mac overlays: Clean up partial swap code

Remove use of linked_ptr, because it's not necessary now that we have
C++11 support. Separate out functions to be a bit easier to read and
follow. Remove unnecessary uses of RectF when Rect would do.

BUG=533681

Review URL: https://codereview.chromium.org/1643043002

Cr-Commit-Position: refs/heads/master@{#372707}
(cherry picked from commit 4d0aa96265cd731ddd74d43821df7feb9c55c1b1)

Mac overlays: Move CALayerPartialDamageTree to its own file

Direct copy-paste, no changes.

BUG=533681
TBR=erikchen

Review URL: https://codereview.chromium.org/1647523002

Cr-Commit-Position: refs/heads/master@{#371871}
(cherry picked from commit 9a9014a549f126db8ad2dc0c0a1dc6bb246423ae)

Mac overlays: Isolate partial swap support

The logic to maintain a CALayer tree for partial swap support is
complicated and should be removed, but that would compromise battery
usage. Once the CoreAniamtion renderer is handling ~99% of the frames,
we can remove it. In the mean time, isolate it.

Create a CALayerPartialDamageTree to track the layer tree of partially
swapped backbuffers. Do this in-place to minimize diffs, so that this
patch can be reasonably reviewed. A follow-on patch will move it to
a separate file.

Slightly reorganize the logic so that CALayerPartialDamageTree matches
the general structure of CALayerTree, where this a pending (not swapped)
tree, a tree for each pending swap, and a tree for what is currently
being displayed.

BUG=533681

Review URL: https://codereview.chromium.org/1640093002

Cr-Commit-Position: refs/heads/master@{#371831}
(cherry picked from commit 8aa00fc2a374fca3e9b9f34d838db7bb5f4fc088)
==========

to

==========
Mac overlays: Clean up partial swap code

Remove use of linked_ptr, because it's not necessary now that we have
C++11 support. Separate out functions to be a bit easier to read and
follow. Remove unnecessary uses of RectF when Rect would do.

BUG=533681

Review URL: https://codereview.chromium.org/1643043002

Cr-Commit-Position: refs/heads/master@{#372707}
(cherry picked from commit 4d0aa96265cd731ddd74d43821df7feb9c55c1b1)

Mac overlays: Move CALayerPartialDamageTree to its own file

Direct copy-paste, no changes.

BUG=533681
TBR=erikchen

Review URL: https://codereview.chromium.org/1647523002

Cr-Commit-Position: refs/heads/master@{#371871}
(cherry picked from commit 9a9014a549f126db8ad2dc0c0a1dc6bb246423ae)

Mac overlays: Isolate partial swap support

The logic to maintain a CALayer tree for partial swap support is
complicated and should be removed, but that would compromise battery
usage. Once the CoreAniamtion renderer is handling ~99% of the frames,
we can remove it. In the mean time, isolate it.

Create a CALayerPartialDamageTree to track the layer tree of partially
swapped backbuffers. Do this in-place to minimize diffs, so that this
patch can be reasonably reviewed. A follow-on patch will move it to
a separate file.

Slightly reorganize the logic so that CALayerPartialDamageTree matches
the general structure of CALayerTree, where this a pending (not swapped)
tree, a tree for each pending swap, and a tree for what is currently
being displayed.

BUG=533681

Review URL: https://codereview.chromium.org/1640093002

Cr-Commit-Position: refs/heads/master@{#371831}
(cherry picked from commit 8aa00fc2a374fca3e9b9f34d838db7bb5f4fc088)

Committed:
https://chromium.googlesource.com/chromium/src/+/04f51c60da4daf1eb228d46954e4...
==========

[ccameron, 2016-02-02 20:35:41]

Committed patchset #1 (id:1) manually as
04f51c60da4daf1eb228d46954e4a8f9d23fbea1.

[erikchen, 2016-02-02 20:40:56]

On 2016/02/02 20:35:41, ccameron wrote:
> Committed patchset #1 (id:1) manually as
> 04f51c60da4daf1eb228d46954e4a8f9d23fbea1.

Why are you merging a refactor to a branch?

[ccameron, 2016-02-02 20:53:18]

On 2016/02/02 20:40:56, erikchen wrote:
> On 2016/02/02 20:35:41, ccameron wrote:
> > Committed patchset #1 (id:1) manually as
> > 04f51c60da4daf1eb228d46954e4a8f9d23fbea1.
> 
> Why are you merging a refactor to a branch?

I'd rather have the branch be in a "feature completed and in known and
cleaned-up state" instead of "enough of the feature merged so that it works on
the branch". The branch already had a big chunk of crbug.com/533681's work, but
was in a halfway point between M48 and M50. This way, if there are any issues,
fixing and merging them will be easy.