Oilpan: fix build after r183834.

Oilpan: fix build after r183834.

TBR=oilpan-reviews,haraken
BUG=423536
NOTRY=true

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

[sof, 2014-10-16 21:17:45]

Please take a look.

[sof, 2014-10-16 21:17:49]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-10-16 21:19:07]

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

[commit-bot: I haz the power, 2014-10-16 21:19:56]

Committed patchset #1 (id:1) as 183837

[dcheng, 2014-10-16 21:25:41]

dcheng@chromium.org changed reviewers:
+ dcheng@chromium.org

[dcheng, 2014-10-16 21:25:42]

https://codereview.chromium.org/640813003/diff/1/Source/web/tests/WebFrameTes...
File Source/web/tests/WebFrameTest.cpp (right):

https://codereview.chromium.org/640813003/diff/1/Source/web/tests/WebFrameTes...
Source/web/tests/WebFrameTest.cpp:6522: RefPtrWillBePersistent<Frame>
childCoreFrame = toCoreFrame(childFrame);
Why not RefPtrWillBeRaw? This is a stack pointer.

[haraken, 2014-10-16 23:19:10]

LGTM

https://codereview.chromium.org/640813003/diff/1/Source/web/tests/WebFrameTes...
File Source/web/tests/WebFrameTest.cpp (right):

https://codereview.chromium.org/640813003/diff/1/Source/web/tests/WebFrameTes...
Source/web/tests/WebFrameTest.cpp:6522: RefPtrWillBePersistent<Frame>
childCoreFrame = toCoreFrame(childFrame);
On 2014/10/16 21:25:42, dcheng wrote:
> Why not RefPtrWillBeRaw? This is a stack pointer.

In this case RefPtrWillBeRawPtr is fine, but we prefer using
RefPtrWillBePersistent in tests because we sometimes have the following code:

RefPtrWillBePersistent<X> x = ...;
Heap::collectAllGarbage();  // Run a precise GC (which assumes that there is no
stack pointer) to collect all objects reliably.
...;  // Test that some object is gone.