Avoid redrawing Display when not damaged.

Avoid redrawing Display when not damaged.

There's no need to do a draw (or swap) if the current frame has no damage, there's no copy request pending, and there's no new latency info.

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

[jbauman, 2015-02-19 00:58:00]

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

[jamesr, 2015-02-19 01:23:49]

tests?

lgtm otherwise

[jbauman, 2015-02-19 23:45:58]

New patchsets have been uploaded after l-g-t-m from jamesr@chromium.org

[jbauman, 2015-02-20 00:07:26]

On 2015/02/19 01:23:49, jamesr wrote:
> tests?
> 
> lgtm otherwise

Ok, added some new tests.

[jamesr, 2015-02-20 00:10:03]

thanks, tests lgtm as well (and I think they'll help us not accidentally delete
one of these cases in the future)

https://codereview.chromium.org/938533002/diff/20001/cc/surfaces/display_unit...
File cc/surfaces/display_unittest.cc (right):

https://codereview.chromium.org/938533002/diff/20001/cc/surfaces/display_unit...
cc/surfaces/display_unittest.cc:79: *called = true;
a lambda would be really nice here, but those don't work with base::Callback. oh
well

[jbauman, 2015-02-20 00:11:57]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2015-02-20 00:13:52]

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

[commit-bot: I haz the power, 2015-02-20 00:27:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-20 00:27:36]

Try jobs failed on following builders:
  android_rel_tests_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_rel_tes...)

[jbauman, 2015-02-20 00:33:15]

New patchsets have been uploaded after l-g-t-m from jamesr@chromium.org

[jbauman, 2015-02-20 00:33:32]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2015-02-20 00:34:11]

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

[commit-bot: I haz the power, 2015-02-20 01:31:00]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-20 01:31:25]

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