Remove RawPtr from bindings/

Remove RawPtr from bindings/

BUG=585328
Committed: https://crrev.com/1a7d8e92dc21d3b58ab778c7997397466c8af1e1
Cr-Commit-Position: refs/heads/master@{#386621}

[keishi, 2016-04-12 07:06:45]

Description was changed from

==========
Remove RawPtr from bindings/core/v8

BUG=585328
==========

to

==========
Remove RawPtr from bindings/

BUG=585328
==========

[keishi, 2016-04-12 07:07:02]

keishi@chromium.org changed reviewers:
+ haraken@chromium.org, oilpan-reviews@chromium.org

[keishi, 2016-04-12 07:07:03]

PTAL

[haraken, 2016-04-12 07:07:49]

LGTM

[keishi, 2016-04-12 07:08:21]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2016-04-12 07:08:27]

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

[sof, 2016-04-12 07:29:53]

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

[sof, 2016-04-12 07:29:54]

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/V8EventListenerList.h (right):

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/V8EventListenerList.h:98:
V8EventListener* wrapperPtr = WrapperType::create(object, isAttribute,
scriptState);
(nit: could just as well reuse |wrapper| here.)

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp (right):

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp:522: return
toV8(RawPtr<HTMLCollection>(items), creationContext, isolate);
Redundant cast?

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/WindowProxyManager.cpp (right):

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/WindowProxyManager.cpp:39:
WindowProxy* isolatedWorldWindowProxy = WindowProxy::create(m_isolate, m_frame,
world);
Shorten out the |isloatedWorldWindowProxy| intermediate.

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/custom/V8EventTargetCustom.cpp
(right):

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/custom/V8EventTargetCustom.cpp:97: {
tangential question - do you know why the extra scope is needed here? Not having
it would avoid having to separate the var decls from their initializations.

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/templates/methods.cpp (right):

https://codereview.chromium.org/1878943003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/templates/methods.cpp:478: MessagePortArray*
ports = new MessagePortArray;
This might conflict with https://codereview.chromium.org/1873323002/

[commit-bot: I haz the power, 2016-04-12 08:16:58]

Description was changed from

==========
Remove RawPtr from bindings/

BUG=585328
==========

to

==========
Remove RawPtr from bindings/

BUG=585328
==========

[commit-bot: I haz the power, 2016-04-12 08:16:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-12 08:18:57]

Description was changed from

==========
Remove RawPtr from bindings/

BUG=585328
==========

to

==========
Remove RawPtr from bindings/

BUG=585328

Committed: https://crrev.com/1a7d8e92dc21d3b58ab778c7997397466c8af1e1
Cr-Commit-Position: refs/heads/master@{#386621}
==========

[commit-bot: I haz the power, 2016-04-12 08:18:58]

Patchset 2 (id:??) landed as
https://crrev.com/1a7d8e92dc21d3b58ab778c7997397466c8af1e1
Cr-Commit-Position: refs/heads/master@{#386621}