Enable paint invalidation container ASSERT

Enable paint invalidation container ASSERT

This ASSERT was disabled with repaint-after-layout. Now that we have
moved invalidation after compositing update, we can enable it. The
ASSERT uncovered a bug in
RenderLayer::enclosingLayerForPaintInvalidation where we were walking
over stacking contexts and not just renderers.

BUG=360286
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178935

[dsinclair, 2014-07-22 19:36:50]

Should this be split into two CLs? One to fix the parent walking and one to
enable the ASSERT? Given the tendency for ASSERTs to cause issues on non-try
bots, it would be nice to not lose the correctness fix on a revert.

[Julien - ping for review, 2014-07-22 20:34:01]

On 2014/07/22 at 19:36:50, dsinclair wrote:
> Should this be split into two CLs? One to fix the parent walking and one to
enable the ASSERT? Given the tendency for ASSERTs to cause issues on non-try
bots, it would be nice to not lose the correctness fix on a revert.

We could but the ASSERT proves the correctness of the fix when I haven't seen
any test changing as we fixed this bug.

[dsinclair, 2014-07-22 20:41:17]

Fair enough, lgtm.

[Julien - ping for review, 2014-07-23 00:00:11]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-07-23 00:01:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jchaffraix@chromium.org/414503002/1

[commit-bot: I haz the power, 2014-07-23 00:04:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/1...)

[commit-bot: I haz the power, 2014-07-23 00:07:05]

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

[commit-bot: I haz the power, 2014-07-23 00:07:06]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/1...)

[Julien - ping for review, 2014-07-23 17:20:28]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-07-23 17:21:10]

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

[commit-bot: I haz the power, 2014-07-23 18:26:03]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/16755)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/32157)

[commit-bot: I haz the power, 2014-07-23 18:46:35]

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

[commit-bot: I haz the power, 2014-07-23 18:46:37]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/32201)

[dsinclair, 2014-07-25 14:46:04]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2014-07-25 14:46:39]

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

[commit-bot: I haz the power, 2014-07-25 15:10:21]

Change committed as 178935