Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(125)

Unified Diff: Source/modules/storage/Storage.cpp

Issue 1055133003: Oilpan: have Storage objects reside on the heap by default. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: trivial cl footprint reduction Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/storage/Storage.cpp
diff --git a/Source/modules/storage/Storage.cpp b/Source/modules/storage/Storage.cpp
index 013be36aa0903828c95010376a2a0cad957a398c..5e6af0e628a8140c4d48efd3e82f1c9d2be47863 100644
--- a/Source/modules/storage/Storage.cpp
+++ b/Source/modules/storage/Storage.cpp
@@ -33,12 +33,12 @@
namespace blink {
-PassRefPtrWillBeRawPtr<Storage> Storage::create(LocalFrame* frame, PassOwnPtrWillBeRawPtr<StorageArea> storageArea)
+Storage* Storage::create(LocalFrame* frame, StorageArea* storageArea)
{
- return adoptRefWillBeNoop(new Storage(frame, storageArea));
+ return new Storage(frame, storageArea);
}
-Storage::Storage(LocalFrame* frame, PassOwnPtrWillBeRawPtr<StorageArea> storageArea)
+Storage::Storage(LocalFrame* frame, StorageArea* storageArea)
: DOMWindowProperty(frame)
, m_storageArea(storageArea)
{
@@ -46,7 +46,9 @@ Storage::Storage(LocalFrame* frame, PassOwnPtrWillBeRawPtr<StorageArea> storageA
ASSERT(m_storageArea);
}
-DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(Storage);
+Storage::~Storage()
+{
+}
String Storage::anonymousIndexedGetter(unsigned index, ExceptionState& exceptionState)
{
@@ -128,4 +130,4 @@ DEFINE_TRACE(Storage)
DOMWindowProperty::trace(visitor);
}
-}
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698