Oilpan: have Storage objects reside on the heap by default.

Source/modules/storage/InspectorDOMStorageAgent.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2013 Samsung Electronics. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 94 matching lines @@
         return;
     m_isEnabled = false;
     m_state->setBoolean(DOMStorageAgentState::domStorageAgentEnabled, false);
     if (StorageNamespaceController* controller = StorageNamespaceController::from(m_page))
         controller->setInspectorAgent(nullptr);
 }
 
 void InspectorDOMStorageAgent::getDOMStorageItems(ErrorString* errorString, const RefPtr<JSONObject>& storageId, RefPtr<TypeBuilder::Array<TypeBuilder::Array<String>>>& items)
 {
     LocalFrame* frame;
-    OwnPtrWillBeRawPtr<StorageArea> storageArea = findStorageArea(errorString, storageId, frame);
+    StorageArea* storageArea = findStorageArea(errorString, storageId, frame);
     if (!storageArea)
         return;
 
     RefPtr<TypeBuilder::Array<TypeBuilder::Array<String>>> storageItems = TypeBuilder::Array<TypeBuilder::Array<String>>::create();
 
     TrackExceptionState exceptionState;
     for (unsigned i = 0; i < storageArea->length(exceptionState, frame); ++i) {
         String name(storageArea->key(i, exceptionState, frame));
         if (hadException(exceptionState, errorString))
             return;
@@ skipping 11 matching lines @@
 static String toErrorString(ExceptionState& exceptionState)
 {
     if (exceptionState.hadException())
         return DOMException::getErrorName(exceptionState.code());
     return "";
 }
 
 void InspectorDOMStorageAgent::setDOMStorageItem(ErrorString* errorString, const RefPtr<JSONObject>& storageId, const String& key, const String& value)
 {
     LocalFrame* frame;
-    OwnPtrWillBeRawPtr<StorageArea> storageArea = findStorageArea(0, storageId, frame);
+    StorageArea* storageArea = findStorageArea(0, storageId, frame);
     if (!storageArea) {
         *errorString = "Storage not found";
         return;
     }
 
     TrackExceptionState exceptionState;
     storageArea->setItem(key, value, exceptionState, frame);
     *errorString = toErrorString(exceptionState);
 }
 
 void InspectorDOMStorageAgent::removeDOMStorageItem(ErrorString* errorString, const RefPtr<JSONObject>& storageId, const String& key)
 {
     LocalFrame* frame;
-    OwnPtrWillBeRawPtr<StorageArea> storageArea = findStorageArea(0, storageId, frame);
+    StorageArea* storageArea = findStorageArea(0, storageId, frame);
     if (!storageArea) {
         *errorString = "Storage not found";
         return;
     }
 
     TrackExceptionState exceptionState;
     storageArea->removeItem(key, exceptionState, frame);
     *errorString = toErrorString(exceptionState);
 }
 
@@ skipping 26 matching lines @@
     for (Frame* frame = inspectedFrame; frame; frame = frame->tree().traverseNext(inspectedFrame)) {
         if (!frame->isLocalFrame())
             continue;
         RefPtr<SecurityOrigin> documentOrigin = toLocalFrame(frame)->document()->securityOrigin();
         if (documentOrigin->toRawString() == originRawString)
             return toLocalFrame(frame);
     }
     return nullptr;
 }
 
-PassOwnPtrWillBeRawPtr<StorageArea> InspectorDOMStorageAgent::findStorageArea(ErrorString* errorString, const RefPtr<JSONObject>& storageId, LocalFrame*& targetFrame)
+StorageArea* InspectorDOMStorageAgent::findStorageArea(ErrorString* errorString, const RefPtr<JSONObject>& storageId, LocalFrame*& targetFrame)
 {
     String securityOrigin;
     bool isLocalStorage = false;
     bool success = storageId->getString("securityOrigin", &securityOrigin);
     if (success)
         success = storageId->getBoolean("isLocalStorage", &isLocalStorage);
     if (!success) {
         if (errorString)
             *errorString = "Invalid storageId format";
         return nullptr;
     }
 
     if (!m_page->mainFrame()->isLocalFrame())
         return nullptr;
 
     LocalFrame* frame = findFrameWithSecurityOrigin(m_page->deprecatedLocalMainFrame(), securityOrigin);
     if (!frame) {
         if (errorString)
             *errorString = "LocalFrame not found for the given security origin";
         return nullptr;
     }
     targetFrame = frame;
 
     if (isLocalStorage)
         return StorageNamespace::localStorageArea(frame->document()->securityOrigin());
     return StorageNamespaceController::from(m_page)->sessionStorage()->storageArea(frame->document()->securityOrigin());
 }
 
 } // namespace blink
-