Oilpan: put ClientRect(List) on the heap.

Source/web/tests/TouchActionTest.cpp

Index: Source/web/tests/TouchActionTest.cpp
diff --git a/Source/web/tests/TouchActionTest.cpp b/Source/web/tests/TouchActionTest.cpp
index 27b76d026b06351fe25df366038e2e4109f1ca2d..a7d15ccb36be0fd4c8d05c3f9a7b1583e5406d1f 100644
--- a/Source/web/tests/TouchActionTest.cpp
+++ b/Source/web/tests/TouchActionTest.cpp
@@ -219,9 +219,9 @@ void TouchActionTest::runTestOnTree(ContainerNode* root, WebView* webView, Touch
         // Note that we don't want the bounding box because our tests sometimes have elements with
         // multiple border boxes with other elements in between. Use the first border box (which
         // we can easily visualize in a browser for debugging).
-        RefPtrWillBePersistent<ClientRectList> rects = element->getClientRects();
+        Persistent<ClientRectList> rects = element->getClientRects();

[haraken, 2015/04/28 00:51:45]

As dcheng pointed out somewhere, if this really needs to be a Persistent
(instead of a raw pointer) for some reason, let's have a comment on this.

         ASSERT_GE(rects->length(), 0u) << failureContext;
-        RefPtrWillBePersistent<ClientRect> r = rects->item(0);
+        Persistent<ClientRect> r = rects->item(0);
         FloatRect clientFloatRect = FloatRect(r->left(), r->top(), r->width(), r->height());
         IntRect clientRect =  enclosedIntRect(clientFloatRect);
         for (int locIdx = 0; locIdx < 3; locIdx++) {