Get rid of hack in invalidatePaintUsingContainer for partially computed

Get rid of hack in invalidatePaintUsingContainer for partially computed
squashing layer state. This is no longer necessary after crrev.com/311813005.

BUG=

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

[dsinclair, 2014-06-13 17:50:04]

https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderObject.cpp (right):

https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderObject.cpp:1515: DisableCompositingQueryAsserts
disabler;
Can this be moved down into the if (v->usesCompositing()) block to limit the
scope?

[chrishtr, 2014-06-13 18:21:28]

On 2014/06/13 17:50:04, dsinclair wrote:
>
https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderObject.cpp (right):
> 
>
https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderObject.cpp:1515: DisableCompositingQueryAsserts
> disabler;
> Can this be moved down into the if (v->usesCompositing()) block to limit the
> scope?

No. Code in RenderView also needs it, and
Is called from this function.

[dsinclair, 2014-06-13 18:23:00]

On 2014/06/13 18:21:28, chrishtr wrote:
> On 2014/06/13 17:50:04, dsinclair wrote:
> >
>
https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
> > File Source/core/rendering/RenderObject.cpp (right):
> > 
> >
>
https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
> > Source/core/rendering/RenderObject.cpp:1515: DisableCompositingQueryAsserts
> > disabler;
> > Can this be moved down into the if (v->usesCompositing()) block to limit the
> > scope?
> 
> No. Code in RenderView also needs it, and
> Is called from this function.

Ah, I failed to understand the comment. Thanks.

[abarth-chromium, 2014-06-13 21:52:06]

LGTM w/ dsinclair's comment addressed.

[chrishtr, 2014-06-13 22:46:49]

https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderObject.cpp (right):

https://codereview.chromium.org/331563005/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderObject.cpp:1515: DisableCompositingQueryAsserts
disabler;
On 2014/06/13 17:50:04, dsinclair wrote:
> Can this be moved down into the if (v->usesCompositing()) block to limit the
> scope?

Per top-level comments, it can't be moved down.

[chrishtr, 2014-06-13 22:47:00]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-13 22:47:50]

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

[commit-bot: I haz the power, 2014-06-14 00:16:04]

Change committed as 176138