Oilpan: move MessagePortArray to the heap.

Oilpan: move MessagePortArray to the heap.

Complete the addition of Oilpan transition types for MessagePort, and
also move its array type to the heap.

R=haraken@chromium.org,adamk@chromium.org
BUG=340522
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176591

[sof, 2014-06-19 13:50:57]

Please take a look.

(Still waiting for verification from a complete Oilpan trybot run.)

[haraken, 2014-06-19 15:31:51]

LGTM

[sof, 2014-06-19 16:45:13]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-06-19 16:46:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/10001

[commit-bot: I haz the power, 2014-06-19 17:44:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-19 17:44:03]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/8034)

[sof, 2014-06-19 20:02:12]

Source/web/ changes needs separate approval

[adamk, 2014-06-19 20:39:12]

lgtm for Source/web

[sof, 2014-06-19 21:05:26]

On 2014/06/19 20:39:12, adamk wrote:
> lgtm for Source/web

many thanks.

[sof, 2014-06-19 21:05:31]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-06-19 21:06:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/10001

[commit-bot: I haz the power, 2014-06-19 21:08:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-19 21:08:22]

Failed to apply patch for Source/bindings/v8/custom/V8MessageEventCustom.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/bindings/v8/custom/V8MessageEventCustom.cpp
  Hunk #1 FAILED at 143.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/bindings/v8/custom/V8MessageEventCustom.cpp.rej

Patch:       Source/bindings/v8/custom/V8MessageEventCustom.cpp
Index: Source/bindings/v8/custom/V8MessageEventCustom.cpp
diff --git a/Source/bindings/v8/custom/V8MessageEventCustom.cpp
b/Source/bindings/v8/custom/V8MessageEventCustom.cpp
index
196d72697fca5022104a46ddd88e0135ed8cf1a9..f6a0464765414b7a6c092ef33efa7a129ed0d1cf
100644
--- a/Source/bindings/v8/custom/V8MessageEventCustom.cpp
+++ b/Source/bindings/v8/custom/V8MessageEventCustom.cpp
@@ -143,12 +143,12 @@ void V8MessageEvent::initMessageEventMethodCustom(const
v8::FunctionCallbackInfo
     TOSTRING_VOID(V8StringResource<>, originArg, info[4]);
     TOSTRING_VOID(V8StringResource<>, lastEventIdArg, info[5]);
     DOMWindow* sourceArg = toDOMWindow(info[6], info.GetIsolate());
-    OwnPtr<MessagePortArray> portArray;
+    OwnPtrWillBeRawPtr<MessagePortArray> portArray = nullptr;
     const int portArrayIndex = 7;
     if (!isUndefinedOrNull(info[portArrayIndex])) {
-        portArray = adoptPtr(new MessagePortArray);
+        portArray = adoptPtrWillBeNoop(new MessagePortArray);
         bool success = false;
-        *portArray = toRefPtrNativeArray<MessagePort,
V8MessagePort>(info[portArrayIndex], portArrayIndex + 1, info.GetIsolate(),
&success);
+        *portArray = toRefPtrWillBeMemberNativeArray<MessagePort,
V8MessagePort>(info[portArrayIndex], portArrayIndex + 1, info.GetIsolate(),
&success);
         if (!success)
             return;
     }

[sof, 2014-06-19 21:42:07]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-06-19 21:43:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/30001

[commit-bot: I haz the power, 2014-06-19 23:04:51]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12759)

[commit-bot: I haz the power, 2014-06-19 23:43:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-19 23:43:49]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12770)

[tkent, 2014-06-20 01:24:05]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-06-20 01:24:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/30001

[commit-bot: I haz the power, 2014-06-20 02:42:51]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12787)

[commit-bot: I haz the power, 2014-06-20 03:41:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-20 03:41:13]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12798)

[tkent, 2014-06-20 05:00:13]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-06-20 05:00:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/30001

[sof, 2014-06-20 05:05:20]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2014-06-20 05:16:16]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-06-20 05:17:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/344833002/30001

[commit-bot: I haz the power, 2014-06-20 05:47:16]

Change committed as 176591