Have ContainerNodeAlgorithms deal with references instead of pointers

Have ContainerNodeAlgorithms deal with references instead of pointers

Have ContainerNodeAlgorithms deal with references instead of pointers
since those can never be null. This makes the code look safer.

R=adamk

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

[adamk, 2013-10-28 21:41:12]

lgtm

https://codereview.chromium.org/49153006/diff/1/Source/core/dom/ContainerNode...
File Source/core/dom/ContainerNode.cpp (right):

https://codereview.chromium.org/49153006/diff/1/Source/core/dom/ContainerNode...
Source/core/dom/ContainerNode.cpp:606: ASSERT(*it);
All these ASSERT(*it) calls make me wonder if we need a some way to have Vector
help us, something like boost::ptr_vector (which stores pointers but exposes
only references).

[Inactive, 2013-10-28 23:03:20]

On 2013/10/28 21:41:12, adamk wrote:
> lgtm
> 
>
https://codereview.chromium.org/49153006/diff/1/Source/core/dom/ContainerNode...
> File Source/core/dom/ContainerNode.cpp (right):
> 
>
https://codereview.chromium.org/49153006/diff/1/Source/core/dom/ContainerNode...
> Source/core/dom/ContainerNode.cpp:606: ASSERT(*it);
> All these ASSERT(*it) calls make me wonder if we need a some way to have
Vector
> help us, something like boost::ptr_vector (which stores pointers but exposes
> only references).

I will take a look and follow , thanks.

[commit-bot: I haz the power, 2013-10-28 23:05:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/49153006/1

[commit-bot: I haz the power, 2013-10-29 00:25:14]

Change committed as 160769