Move quota module to oilpan

Source/modules/quota/DOMWindowQuota.h

Index: Source/modules/quota/DOMWindowQuota.h
diff --git a/Source/modules/quota/DOMWindowQuota.h b/Source/modules/quota/DOMWindowQuota.h
index dab72266c7042e8c04abd0133912ace93e3e3291..5e4cfa4c913356aaafced581ed5946f821d41996 100644
--- a/Source/modules/quota/DOMWindowQuota.h
+++ b/Source/modules/quota/DOMWindowQuota.h
@@ -32,6 +32,7 @@
 #define DOMWindowQuota_h
 
 #include "core/frame/DOMWindowProperty.h"
+#include "heap/Handle.h"
 #include "platform/Supplementable.h"
 
 namespace WebCore {
@@ -41,7 +42,7 @@ class DOMWindow;
 
 class DOMWindowQuota FINAL : public Supplement<DOMWindow>, public DOMWindowProperty {
 public:
-    virtual ~DOMWindowQuota();
+    virtual ~DOMWindowQuota() { }

[tkent, 2014/02/10 08:27:40]

Why do you change this?

[sof, 2014/02/10 08:32:13]

Why wouldn't you want to see in the header file that the destructor is empty and
that (potentially) a *GarbageCollectedFinalized is not needed?

i.e., what do you gain from not inlining the empty destructor here (and
elsewhere)?

[tkent, 2014/02/10 08:41:57]

This CL should focus on Oilapan transition.  Changes unnecessary for Oilpan
would be noise.

Also, virtual destructors can not be inlined.  This change has no performance
benefit and makes build performance slightly worse.

[sof, 2014/02/10 08:45:09]

Hmm.. ok.
I don't think performance was the argument for it here....

[sof, 2014/02/10 09:07:19]

Reverted.

     static DOMWindowQuota* from(DOMWindow*);
     static DeprecatedStorageInfo* webkitStorageInfo(DOMWindow*);
     DeprecatedStorageInfo* webkitStorageInfo() const;
@@ -50,7 +51,7 @@ private:
     explicit DOMWindowQuota(DOMWindow*);
     static const char* supplementName();
 
-    mutable RefPtr<DeprecatedStorageInfo> m_storageInfo;
+    mutable RefPtrWillBePersistent<DeprecatedStorageInfo> m_storageInfo;
 };
 
 } // namespace WebCore