Oilpan: put ClientRect(List) on the heap.

Source/web/tests/TouchActionTest.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 201 matching lines @@
             failureContext.append(element->firstChild()->textContent(false).stripWhiteSpace().ascii().data());
             failureContext.append("\"");
         } else {
             failureContext += "<missing ID>";
         }
 
         // Run each test three times at different positions in the element.
         // 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++) {
             IntPoint clientPoint;
             std::stringstream contextStream;
             contextStream << failureContext << " (";
             switch (locIdx) {
             case 0:
                 clientPoint = clientRect.center();
                 contextStream << "center";
@@ skipping 105 matching lines @@
 {
     runShadowDOMTest("touch-action-shadow-dom.html");
 }
 
 TEST_F(TouchActionTest, Pan)
 {
     runTouchActionTest("touch-action-pan.html");
 }
 
 }