IDL: Use raw pointers instead of RawPtr<> for rvalue types

IDL: Use raw pointers instead of RawPtr<> for rvalue types

For unconditionally garbage collected types, use of RawPtr<> is
unnecessary.

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

[Jens Widell, 2014-11-13 16:57:35]

Patchset #1 (id:1) has been deleted

[Jens Widell, 2014-11-13 17:04:10]

jl@opera.com changed reviewers:
+ haraken@chromium.org, sigbjornf@opera.com

[Jens Widell, 2014-11-13 17:04:11]

PTAL

As suggested in https://codereview.chromium.org/714123002/.

Getting rid of all plain use of RawPtr<> in generated code would be possible,
but a bit more involved. The rest are "internal" though, so doesn't really
matter.

This change affects: dictionary member accessors, union member accessors, and
the argument types of private script methods and attribute setters. (Except no
such dictionary members or private script functions currently exist to be
affected.)

[sof, 2014-11-13 17:40:26]

(non-owner) lgtm.

(Kick off some trybots to verify buildability?)

[haraken, 2014-11-14 00:35:26]

LGTM

[haraken, 2014-11-14 00:38:58]

> Getting rid of all plain use of RawPtr<> in generated code would be possible,
> but a bit more involved. The rest are "internal" though, so doesn't really
> matter.

Yeah, we intentionally use RawPtr<T> instead of T* in some cases to avoid wrong
overload resolution of toV8() etc. See
https://codereview.chromium.org/257503003/#msg1

[Jens Widell, 2014-11-17 07:09:49]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-11-17 07:10:12]

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

[commit-bot: I haz the power, 2014-11-17 09:23:10]

Committed patchset #1 (id:20001) as 185425