Remove the CompositedLayerMappingPtr smart ptr.

Remove the CompositedLayerMappingPtr smart ptr.

All this class did was prevent callers from null pointer checking the
object, but all hasCompositedLayerMapping() does is null check it so it's
not clear what value this is adding, lets remove it.

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

[abarth-chromium, 2014-08-08 15:34:20]

LGTM

https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderLayer.h (right):

https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderLayer.h:318: bool hasCompositedLayerMapping() const
{ return m_compositedLayerMapping.get(); }
Shawn's idea was to make all the call sites that wanted to null-check
m_compositedLayerMapping explicitly call a function that was greppable in order
to make it easier to implement squashing.

[chrishtr, 2014-08-08 16:11:42]

lgtm

https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderLayer.h (right):

https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderLayer.h:318: bool hasCompositedLayerMapping() const
{ return m_compositedLayerMapping.get(); }
On 2014/08/08 15:34:19, abarth wrote:
> Shawn's idea was to make all the call sites that wanted to null-check
> m_compositedLayerMapping explicitly call a function that was greppable in
order
> to make it easier to implement squashing.

Let's get rid of hasCompositedLayerMapping in this CL.

[esprehn, 2014-08-09 05:06:23]

On 2014/08/08 at 16:11:42, chrishtr wrote:
> lgtm
> 
>
https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderLayer.h (right):
> 
>
https://codereview.chromium.org/457473002/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderLayer.h:318: bool hasCompositedLayerMapping()
const { return m_compositedLayerMapping.get(); }
> On 2014/08/08 15:34:19, abarth wrote:
> > Shawn's idea was to make all the call sites that wanted to null-check
> > m_compositedLayerMapping explicitly call a function that was greppable in
order
> > to make it easier to implement squashing.
> 
> Let's get rid of hasCompositedLayerMapping in this CL.

I'll do that in the next CL.

[esprehn, 2014-08-09 05:06:26]

The CQ bit was checked by esprehn@chromium.org

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

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

[commit-bot: I haz the power, 2014-08-09 06:06:13]

Change committed as 179883