Remove uses of PaintBehaviorFlattenCompositingLayers

Remove uses of PaintBehaviorFlattenCompositingLayers

R=ojan@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/c5d377fbb78329be42a7c1d5a95edb893be63fad

[rafaelw, 2014-11-29 04:42:15]

rafaelw@chromium.org changed reviewers:
+ ojan@chromium.org

[ojan, 2014-12-01 19:00:38]

lgtm

https://codereview.chromium.org/766943003/diff/1/sky/engine/core/rendering/Re...
File sky/engine/core/rendering/RenderBox.cpp (right):

https://codereview.chromium.org/766943003/diff/1/sky/engine/core/rendering/Re...
sky/engine/core/rendering/RenderBox.cpp:925:
paintNinePieceImage(paintInfo.context, paintRect, style(),
style()->maskBoxImage(), CompositeSourceOver);
Do paintFillLayers and/or paintNinePieceImage ever take a value other than
CompositeSourceOver now? Maybe you can get rid of that argument entirely?

[rafaelw, 2014-12-01 19:26:11]

https://codereview.chromium.org/766943003/diff/1/sky/engine/core/rendering/Re...
File sky/engine/core/rendering/RenderBox.cpp (right):

https://codereview.chromium.org/766943003/diff/1/sky/engine/core/rendering/Re...
sky/engine/core/rendering/RenderBox.cpp:925:
paintNinePieceImage(paintInfo.context, paintRect, style(),
style()->maskBoxImage(), CompositeSourceOver);
On 2014/12/01 19:00:38, ojan wrote:
> Do paintFillLayers and/or paintNinePieceImage ever take a value other than
> CompositeSourceOver now? Maybe you can get rid of that argument entirely?

Nope. done. Also paintFillLayer

[ojan, 2014-12-01 19:30:30]

lgtm

I expect we can simplify some of GraphicsContext as well now to pass fewer
CompositeOperators around, but maybe not. In either case, that would be best to
do in a separate patch.

[rafaelw, 2014-12-01 19:49:37]

Committed patchset #3 (id:40001) manually as
c5d377fbb78329be42a7c1d5a95edb893be63fad.