Oilpan: Move core/clipboard/ to oilpan's heap

Source/web/WebDragData.cpp

 /*
  * Copyright (C) 2009 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 14 matching lines @@
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 
 #include "core/clipboard/DataObject.h"
 #include "core/clipboard/DataTransferItem.h"
+#include "heap/Handle.h"
 #include "modules/filesystem/DraggedIsolatedFileSystem.h"
 #include "platform/clipboard/ClipboardMimeTypes.h"
 #include "public/platform/WebData.h"
 #include "public/platform/WebDragData.h"
 #include "public/platform/WebString.h"
 #include "public/platform/WebURL.h"
 #include "public/platform/WebVector.h"
 #include "wtf/HashMap.h"
 #include "wtf/PassRefPtr.h"
 
 using namespace WebCore;
 
 namespace blink {
 
-class WebDragDataPrivate : public DataObject {
-};
+WebDragData::WebDragData(const WebDragData& object)
+{
+    assign(object);
+}
+
+WebDragData::~WebDragData()
+{
+    reset();
+}
+
+WebDragData& WebDragData::operator=(const WebDragData& object)
+{
+    assign(object);
+    return *this;
+}
+
+bool WebDragData::isNull() const
+{
+    return m_private.isNull();
+}
 
 void WebDragData::initialize()
 {
-    assign(static_cast<WebDragDataPrivate*>(DataObject::create().leakRef()));
+    m_private = DataObject::create();
 }
 
 void WebDragData::reset()
 {
-    assign(0);
+    m_private.reset();
 }
 
 void WebDragData::assign(const WebDragData& other)
 {
-    WebDragDataPrivate* p = const_cast<WebDragDataPrivate*>(other.m_private);
-    if (p)
-        p->ref();
-    assign(p);
+    m_private = other.m_private;
+}
+
+WebDragData::WebDragData(const PassRefPtrWillBeRawPtr<WebCore::DataObject>& object)
+{
+    assign(object);
+}
+
+WebDragData& WebDragData::operator=(const PassRefPtrWillBeRawPtr<WebCore::DataObject>& object)
+{
+    assign(object);
+    return *this;
+}
+
+DataObject* WebDragData::getValue() const

[sof, 2014/02/19 08:30:24]

I'm not sure getValue() is worth having in this setting; what do you think?

[haraken, 2014/02/19 08:37:12]

Yeah, this might be unnecessary.

In the experimental branch, 'operator WTF::PassRefPtr<WebCore::DataObject>()'
caused some issue, so we introduced getValue() to remove the implicit
conversion. If I can remove the implicit conversion without introducing
getValue(), I'm happy to remove getValue(). Let me try.

+{
+    return m_private.get();
+}
+
+void WebDragData::ensureMutable()
+{
+    ASSERT(!isNull());
 }
 
 WebVector<WebDragData::Item> WebDragData::items() const
 {
     Vector<Item> itemList;
-    for (size_t i = 0; i < m_private->length(); ++i) {
-        DataObjectItem* originalItem = m_private->item(i).get();
+    for (size_t i = 0; i < getValue()->length(); ++i) {
+        DataObjectItem* originalItem = getValue()->item(i).get();
         WebDragData::Item item;
         if (originalItem->kind() == DataObjectItem::StringKind) {
             item.storageType = Item::StorageTypeString;
             item.stringType = originalItem->type();
             item.stringData = originalItem->getAsString();
         } else if (originalItem->kind() == DataObjectItem::FileKind) {
             if (originalItem->sharedBuffer()) {
                 item.storageType = Item::StorageTypeBinaryData;
                 item.binaryData = originalItem->sharedBuffer();
             } else if (originalItem->isFilename()) {
@@ skipping 11 matching lines @@
             ASSERT_NOT_REACHED();
         item.title = originalItem->title();
         item.baseURL = originalItem->baseURL();
         itemList.append(item);
     }
     return itemList;
 }
 
 void WebDragData::setItems(const WebVector<Item>& itemList)
 {
-    m_private->clearAll();
+    getValue()->clearAll();
     for (size_t i = 0; i < itemList.size(); ++i)
         addItem(itemList[i]);
 }
 
 void WebDragData::addItem(const Item& item)
 {
     ensureMutable();
     switch (item.storageType) {
     case Item::StorageTypeString:
         if (String(item.stringType) == mimeTypeTextURIList)
-            m_private->setURLAndTitle(item.stringData, item.title);
+            getValue()->setURLAndTitle(item.stringData, item.title);
         else if (String(item.stringType) == mimeTypeTextHTML)
-            m_private->setHTMLAndBaseURL(item.stringData, item.baseURL);
+            getValue()->setHTMLAndBaseURL(item.stringData, item.baseURL);
         else
-            m_private->setData(item.stringType, item.stringData);
+            getValue()->setData(item.stringType, item.stringData);
         return;
     case Item::StorageTypeFilename:
-        m_private->addFilename(item.filenameData, item.displayNameData);
+        getValue()->addFilename(item.filenameData, item.displayNameData);
         return;
     case Item::StorageTypeBinaryData:
         // This should never happen when dragging in.
         ASSERT_NOT_REACHED();
     }
 }
 
 WebString WebDragData::filesystemId() const
 {
     ASSERT(!isNull());
-    DraggedIsolatedFileSystem* filesystem = DraggedIsolatedFileSystem::from(m_private);
+    DraggedIsolatedFileSystem* filesystem = DraggedIsolatedFileSystem::from(getValue());
     if (filesystem)
         return filesystem->filesystemId();
     return WebString();
 }
 
 void WebDragData::setFilesystemId(const WebString& filesystemId)
 {
     // The ID is an opaque string, given by and validated by chromium port.
     ensureMutable();
-    DraggedIsolatedFileSystem::provideTo(m_private, DraggedIsolatedFileSystem::supplementName(), DraggedIsolatedFileSystem::create(filesystemId));
-}
-
-WebDragData::WebDragData(const WTF::PassRefPtr<WebCore::DataObject>& data)
-    : m_private(static_cast<WebDragDataPrivate*>(data.leakRef()))
-{
-}
-
-WebDragData& WebDragData::operator=(const WTF::PassRefPtr<WebCore::DataObject>& data)
-{
-    assign(static_cast<WebDragDataPrivate*>(data.leakRef()));
-    return *this;
-}
-
-WebDragData::operator WTF::PassRefPtr<WebCore::DataObject>() const
-{
-    return PassRefPtr<DataObject>(const_cast<WebDragDataPrivate*>(m_private));
-}
-
-void WebDragData::assign(WebDragDataPrivate* p)
-{
-    // p is already ref'd for us by the caller
-    if (m_private)
-        m_private->deref();
-    m_private = p;
-}
-
-void WebDragData::ensureMutable()
-{
-    ASSERT(!isNull());
-    ASSERT(m_private->hasOneRef());
+    DraggedIsolatedFileSystem::provideTo(getValue(), DraggedIsolatedFileSystem::supplementName(), DraggedIsolatedFileSystem::create(filesystemId));
 }
 
 } // namespace blink