DCLayerOverlay should damage entire frame on resize.

DCLayerOverlay should damage entire frame on resize.

The output surface (and SetDamageRectCHROMIUM) requires that the entire
output surface be redrawn on resize. So don't attempt to subtract out
damage in this case. Also, the damage rect must always be contained
within the output rect, so clip it to that.

BUG=678800
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2754663003
Cr-Commit-Position: refs/heads/master@{#457632}
Committed: https://chromium.googlesource.com/chromium/src/+/8f0264a9eb0620b67b746a44b5856eea61245f77

[jbauman, 2017-03-15 22:35:34]

Description was changed from

==========
DCLayerOverlay should damage entire frame on resize.

The output surface (and SetDamageRectCHROMIUM) requires that the entire
output surface be redrawn on resize. So don't attempt to subtract out
damage in this case. Also, the damage rect must always be contained
within the output rect, so clip it to that.

BUG=678800
==========

to

==========
DCLayerOverlay should damage entire frame on resize.

The output surface (and SetDamageRectCHROMIUM) requires that the entire
output surface be redrawn on resize. So don't attempt to subtract out
damage in this case. Also, the damage rect must always be contained
within the output rect, so clip it to that.

BUG=678800
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[jbauman, 2017-03-15 22:35:40]

The CQ bit was checked by jbauman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-15 22:36:08]

Dry run: 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-15 22:42:28]

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

[commit-bot: I haz the power, 2017-03-15 22:42:30]

Dry run: Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_trusty_blink_rel/b...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jbauman, 2017-03-15 23:34:54]

The CQ bit was checked by jbauman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-15 23:35:43]

Dry run: 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-16 00:14:16]

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

[commit-bot: I haz the power, 2017-03-16 00:14:17]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[jbauman, 2017-03-16 22:52:01]

The CQ bit was checked by jbauman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-16 22:52:41]

Dry run: 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-16 23:48:16]

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

[commit-bot: I haz the power, 2017-03-16 23:48:17]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-03-16 23:57:21]

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

[ccameron, 2017-03-17 00:02:09]

lgtm

[jbauman, 2017-03-17 00:17:09]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2017-03-17 00:18: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-17 00:47:18]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489709829735230,
"parent_rev": "9e2fe00fdef6b5786d8077ceae63d92587b8110b", "commit_rev":
"8f0264a9eb0620b67b746a44b5856eea61245f77"}

[commit-bot: I haz the power, 2017-03-17 00:48:04]

Description was changed from

==========
DCLayerOverlay should damage entire frame on resize.

The output surface (and SetDamageRectCHROMIUM) requires that the entire
output surface be redrawn on resize. So don't attempt to subtract out
damage in this case. Also, the damage rect must always be contained
within the output rect, so clip it to that.

BUG=678800
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
DCLayerOverlay should damage entire frame on resize.

The output surface (and SetDamageRectCHROMIUM) requires that the entire
output surface be redrawn on resize. So don't attempt to subtract out
damage in this case. Also, the damage rect must always be contained
within the output rect, so clip it to that.

BUG=678800
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2754663003
Cr-Commit-Position: refs/heads/master@{#457632}
Committed:
https://chromium.googlesource.com/chromium/src/+/8f0264a9eb0620b67b746a44b585...
==========

[commit-bot: I haz the power, 2017-03-17 00:48:05]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/8f0264a9eb0620b67b746a44b585...