Remove DeprecatedPaintLayerFragment::outlineRect

Remove DeprecatedPaintLayerFragment::outlineRect

The field was redundant with DeprecatedPaintLayerFragment::backgroundRect.
The reason is that the outline rect was computed from the backgroundRect,
with the difference that backgroundRect was intersected with the visual
overflow rect. As the visual overflow contains the outline, the
backgroundRect should be enough for outline clipping.

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

[Julien - ping for review, 2015-09-17 14:23:16]

jchaffraix@chromium.org changed reviewers:
+ mstensho@opera.com, pdr@chromium.org

[Julien - ping for review, 2015-09-17 14:23:16]

+Morten as an FYI as it touches fragments (feel free to review, you boisterous
vikin'!) 
+Philip for main review. It's unclear if it's up your alley but there is no
clear owner for this class :(

[mstensho (USE GERRIT), 2015-09-17 14:45:07]

The fewer rects the merrier! lgtm from Europe.

https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectations
File LayoutTests/TestExpectations (right):

https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectation...
LayoutTests/TestExpectations:1114: 
That's the blankest shiniest line I've ever seen!

[pdr., 2015-09-18 01:56:32]

On 2015/09/17 at 14:45:07, mstensho wrote:
> The fewer rects the merrier! lgtm from Europe.
> 
> https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectations
> File LayoutTests/TestExpectations (right):
> 
>
https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectation...
> LayoutTests/TestExpectations:1114: 
> That's the blankest shiniest line I've ever seen!

LGTM2

[Julien - ping for review, 2015-09-19 01:59:46]

https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectations
File LayoutTests/TestExpectations (right):

https://codereview.chromium.org/1349473003/diff/1/LayoutTests/TestExpectation...
LayoutTests/TestExpectations:1114: 
On 2015/09/17 at 14:45:07, mstensho wrote:
> That's the blankest shiniest line I've ever seen!

Shiny, shiny, shiny *_*

[Julien - ping for review, 2015-09-21 15:46:17]

The CQ bit was checked by jchaffraix@chromium.org

[Julien - ping for review, 2015-09-21 15:46:20]

The patchset sent to the CQ was uploaded after l-g-t-m from mstensho@opera.com
Link to the patchset: https://codereview.chromium.org/1349473003/#ps20001
(title: "Removed line, added rebaselines.")

[commit-bot: I haz the power, 2015-09-21 15:46:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1349473003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1349473003/20001

[commit-bot: I haz the power, 2015-09-21 17:33:04]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=202587