Don't layout when isSelfPaintingLayer() changes

Don't layout when isSelfPaintingLayer() changes

I can't find any reason to do a layout when the self painting bit
changes. The code was added by hyatt a long time ago in
https://trac.webkit.org/changeset/43657 but the test he added still
passes without that code (as do the rest of the tests).

Lets just delete this layout code.

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

[abarth-chromium, 2014-05-08 01:37:50]

LGTM

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

https://codereview.chromium.org/275563002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderLayerModelObject.cpp:166: repaint();
Do any tests fail if you just delete this line?

[esprehn, 2014-05-08 01:39:04]

On 2014/05/08 01:37:50, abarth wrote:
> LGTM
> 
>
https://codereview.chromium.org/275563002/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderLayerModelObject.cpp (right):
> 
>
https://codereview.chromium.org/275563002/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderLayerModelObject.cpp:166: repaint();
> Do any tests fail if you just delete this line?

That's a good question, I didn't try it.

[esprehn, 2014-05-08 07:21:26]

Yeah it seems everything passes without that code, lets delete it.

[esprehn, 2014-05-08 07:21:35]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2014-05-08 07:22:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/esprehn@chromium.org/275563002/40001

[commit-bot: I haz the power, 2014-05-08 08:00:44]

Change committed as 173614