cc: Invalidate all new recorded pixels when a layer grows.

cc: Invalidate all new recorded pixels when a layer grows.

Previously we invalidated only new recorded pixels on tiles that
resized but ignored new recording tiles. The problem with this is that
if the old layer size is on the boundary of recording tiles, then we
invalidate nothing. But there may be raster tiles that would like to
use the new recording tiles, and currently have uninitialized content
in them, but they don't get invalidated to hear about it.

R=vmpstr
BUG=411774
Committed: https://crrev.com/d0a11b12e7fcc37409d2fcb35c8a59f44d82b5d5
Cr-Commit-Position: refs/heads/master@{#294203}

[vmpstr, 2014-09-09 22:43:00]

lgtm. Thanks for tracking this down.

https://codereview.chromium.org/556973003/diff/1/cc/resources/picture_pile_un...
File cc/resources/picture_pile_unittest.cc (right):

https://codereview.chromium.org/556973003/diff/1/cc/resources/picture_pile_un...
cc/resources/picture_pile_unittest.cc:475: // We invalidated any new pixels.
nit: That comment doesn't make sense to me, can you reword?

[danakj, 2014-09-10 16:52:39]

The CQ bit was checked by danakj@chromium.org

[danakj, 2014-09-10 16:52:49]

Thanks!

https://codereview.chromium.org/556973003/diff/1/cc/resources/picture_pile_un...
File cc/resources/picture_pile_unittest.cc (right):

https://codereview.chromium.org/556973003/diff/1/cc/resources/picture_pile_un...
cc/resources/picture_pile_unittest.cc:475: // We invalidated any new pixels.
On 2014/09/09 22:43:00, vmpstr wrote:
> nit: That comment doesn't make sense to me, can you reword?

Done. s/any new pixels/all new pixels in the recording/

[commit-bot: I haz the power, 2014-09-10 16:52:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/danakj@chromium.org/556973003/20001

[commit-bot: I haz the power, 2014-09-10 18:11:48]

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

[commit-bot: I haz the power, 2014-09-10 18:11:49]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[danakj, 2014-09-10 18:21:26]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-09-10 18:22:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/danakj@chromium.org/556973003/20001

[commit-bot: I haz the power, 2014-09-10 19:03:08]

Committed patchset #2 (id:20001) as 5bef4a0ed8bf61340f8c60c7e3febc01227ce6ce

[commit-bot: I haz the power, 2014-09-10 19:19:37]

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