Always force swap for SurfaceDisplayOutputSurface.

Always force swap for SurfaceDisplayOutputSurface.

Force reclaim clears out the Surface's copy of the delegated frame, so a swap is always necessary to restore its contents.

Committed: https://crrev.com/e3c79bdf0c63955eeeef57621bfb07ec19fb7935
Cr-Commit-Position: refs/heads/master@{#314492}

[jbauman, 2015-02-04 00:22:43]

jbauman@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2015-02-04 01:30:18]

Maybe I'm being mislead about the setting name, but does this mean we swap even
when we have no computed damage?  Do we force reclaim resources every frame?

[jbauman, 2015-02-04 01:32:34]

On 2015/02/04 01:30:18, jamesr wrote:
> Maybe I'm being mislead about the setting name, but does this mean we swap
even
> when we have no computed damage?  Do we force reclaim resources every frame?

The compositor should swap after every commit, but the Display doesn't need to,
as there's no damage.

[jamesr, 2015-02-04 02:30:22]

Right, gotcha.  This is confusing.  IMHO we should decouple the "OutputSurface
that actually just gives you a context to use for delegation and a place to
submit frame data" from the "OutputSurface that is a Display" interfaces.

lgtm

[jbauman, 2015-02-04 02:31:16]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2015-02-04 02:31:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/886633007/1

[commit-bot: I haz the power, 2015-02-04 03:34:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-04 03:35:40]

Patchset 1 (id:??) landed as
https://crrev.com/e3c79bdf0c63955eeeef57621bfb07ec19fb7935
Cr-Commit-Position: refs/heads/master@{#314492}