Add layer bounds to compositing_dislay_item

Add layer bounds to compositing_dislay_item

This patch adds an SkRect for bounds to compositing_display_item and
uses it for setting the canvas layer bounds. With this patch and the
associated blink-side patch [1] we pass the following additional
layout tests:
LayoutTests/svg/zoom/page/zoom-mask-with-percentages.svg
LayoutTests/svg/transforms/text-with-mask-with-svg-transform.svg
LayoutTests/svg/custom/absolute-sized-content-with-resources.xhtml
LayoutTests/svg/custom/mask-excessive-malloc.svg
LayoutTests/svg/batik/masking/maskRegions.svg

[1] https://codereview.chromium.org/960483002

Committed: https://crrev.com/18fc53626c407d76c22c91132cabaa7a0923ed62
Cr-Commit-Position: refs/heads/master@{#318127}

[pdr., 2015-02-25 06:16:21]

pdr@chromium.org changed reviewers:
+ ajuma@chromium.org

[ajuma, 2015-02-25 14:22:08]

lgtm

[pdr., 2015-02-25 18:26:32]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2015-02-25 18:27:38]

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

[pdr., 2015-02-25 20:42:43]

On 2015/02/25 at 18:27:38, commit-bot wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/952123002/20001

Looks like we were failing to compile some places due to a pointer -> bool
conversion. I updated compositing_display_item.cc to make this conversion
explicit.

[pdr., 2015-02-25 20:43:05]

The CQ bit was checked by pdr@chromium.org

[pdr., 2015-02-25 20:43:05]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/952123002/#ps40001 (title:
"Remove implicit pointer to bool conversion")

[ajuma, 2015-02-25 20:44:30]

https://codereview.chromium.org/952123002/diff/40001/cc/resources/compositing...
File cc/resources/compositing_display_item.cc (right):

https://codereview.chromium.org/952123002/diff/40001/cc/resources/compositing...
cc/resources/compositing_display_item.cc:22: has_bounds_(bounds != nullptr),
The usual pattern to workaround that error is to use '!!bounds'.

[commit-bot: I haz the power, 2015-02-25 20:44:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/952123002/40001

[pdr., 2015-02-25 20:54:58]

The CQ bit was unchecked by pdr@chromium.org

[pdr., 2015-02-25 20:56:20]

On 2015/02/25 at 20:44:30, ajuma wrote:
>
https://codereview.chromium.org/952123002/diff/40001/cc/resources/compositing...
> File cc/resources/compositing_display_item.cc (right):
> 
>
https://codereview.chromium.org/952123002/diff/40001/cc/resources/compositing...
> cc/resources/compositing_display_item.cc:22: has_bounds_(bounds != nullptr),
> The usual pattern to workaround that error is to use '!!bounds'.

Ahh thanks! Updated.

[pdr., 2015-02-25 20:56:23]

The CQ bit was checked by pdr@chromium.org

[pdr., 2015-02-25 20:56:23]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/952123002/#ps60001 (title:
"!!bounds!")

[commit-bot: I haz the power, 2015-02-25 20:57:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/952123002/60001

[commit-bot: I haz the power, 2015-02-25 22:29:33]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-25 22:45:00]

Patchset 4 (id:??) landed as
https://crrev.com/18fc53626c407d76c22c91132cabaa7a0923ed62
Cr-Commit-Position: refs/heads/master@{#318127}