GridPainter::paintChild should take RenderBox reference

GridPainter::paintChild should take RenderBox reference

GridPainter method paintChild() take pointer to RenderBox instance but
the body of this method assumes that the object will never be NULL.
We should replace it to use RenderBox references. This somehow got
missed in the previous revision of converting pointers to references.

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

[Sunil Ratnu, 2014-09-19 06:39:40]

sunil.ratnu@samsung.com changed reviewers:
+ esprehn@chromium.org, jchaffraix@chromium.org

[Sunil Ratnu, 2014-09-19 06:39:41]

A minor change. Please review this. Thanks!

[Sunil Ratnu, 2014-09-23 19:50:03]

On 2014/09/19 06:39:41, Sunil Ratnu wrote:
> A minor change. Please review this. Thanks!

Gentle ping!

[mstensho (USE GERRIT), 2014-09-25 06:52:10]

mstensho@opera.com changed reviewers:
+ mstensho@opera.com

[mstensho (USE GERRIT), 2014-09-25 06:52:11]

RenderGrid painting code has just been moved to GridPainter [1], so you need to
update your patch. BTW: the paintChild() declaration was accidentally left
behind in that code move patch, but I've filed a CL for that [2].

[1] https://codereview.chromium.org/594293002/
[2] https://codereview.chromium.org/601113002/

[Sunil Ratnu, 2014-09-25 07:06:42]

On 2014/09/25 06:52:11, mstensho wrote:
> RenderGrid painting code has just been moved to GridPainter [1], so you need
to
> update your patch. BTW: the paintChild() declaration was accidentally left
> behind in that code move patch, but I've filed a CL for that [2].
> 
> [1] https://codereview.chromium.org/594293002/
> [2] https://codereview.chromium.org/601113002/

Thanks for that update mstensho. This patch has been pending for a long time :(.
I've rebased to the latest now.

[esprehn, 2014-09-26 04:00:53]

lgtm

[Sunil Ratnu, 2014-09-26 05:02:30]

The CQ bit was checked by sunil.ratnu@samsung.com

[commit-bot: I haz the power, 2014-09-26 05:03:08]

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

[commit-bot: I haz the power, 2014-09-26 06:53:29]

Committed patchset #2 (id:20001) as 182738