Have NodeChildRemovalTracker constructor take a reference

Have NodeChildRemovalTracker constructor take a reference

Have NodeChildRemovalTracker constructor take a reference instead of
a pointer and update NodeChildRemovalTracker::node() to return a
reference, as those pointers could not be null.

This makes the calling code look safer.

R=adamk

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

[adamk, 2013-10-30 21:11:35]

lgtm

[commit-bot: I haz the power, 2013-10-30 21:12:54]

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

[tfarina, 2013-10-30 21:58:50]

Shouldn't Blink adhere to Google C++ Style guide, now that it isn't tied to
Apple?

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Refere...

These reference changes are making me nervous!

[Inactive, 2013-10-30 22:01:19]

On 2013/10/30 21:58:50, tfarina wrote:
> Shouldn't Blink adhere to Google C++ Style guide, now that it isn't tied to
> Apple?
> 
>
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Refere...
> 
> These reference changes are making me nervous!

There was a long discussion about this on blink-dev. I thought we reached the
agreement that we would use references in Blink?

[commit-bot: I haz the power, 2013-10-30 23:33:19]

List of reviewers changed. tfarina@chromium.org did a drive-by without LGTM'ing!

[commit-bot: I haz the power, 2013-10-30 23:53:52]

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

[commit-bot: I haz the power, 2013-10-30 23:56:01]

Change committed as 160988