Attachment 224699 Details for Bug 129078 – Patch

[patch] Patch

bug-129078-20140220111615.patch (text/plain), 2.81 KB, created by Gyuyoung Kim on 2014-02-19 18:16:16 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Gyuyoung Kim

Created: 2014-02-19 18:16:16 PST

Size: 2.81 KB

patch

obsolete

>Subversion Revision: 164362
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 4d08be5da5815688f6850b27c0497821284d0ae6..3956c57c58b229b59e2cbc723ee812c1f832a00c 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,17 @@
>+2014-02-19  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
>+
>+        Start to use std::unique_ptr for DragImageLoader
>+        https://bugs.webkit.org/show_bug.cgi?id=129078
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        As a step to use std::unique_ptr, this patch replace uses of OwnPtr and PassOwnPtr
>+        in the DragImageLoader class with std::unique_ptr.
>+
>+        * dom/Clipboard.cpp:
>+        (WebCore::Clipboard::setDragImage):
>+        * dom/Clipboard.h:
>+
> 2014-02-19  Xabier Rodriguez Calvar  <calvaris@igalia.com>
> 
>         [GStreamer] the GstPlayFlags enum diverged from upstream
>diff --git a/Source/WebCore/dom/Clipboard.cpp b/Source/WebCore/dom/Clipboard.cpp
>index 30d13dd19813f7f3b2212179c34d510b9615af68..de17fe59a8a3a82ec40c7e89fc528c7b26dc9bda 100644
>--- a/Source/WebCore/dom/Clipboard.cpp
>+++ b/Source/WebCore/dom/Clipboard.cpp
>@@ -44,12 +44,11 @@ namespace WebCore {
> class DragImageLoader final : private CachedImageClient {
>     WTF_MAKE_FAST_ALLOCATED;
> public:
>-    static PassOwnPtr<DragImageLoader> create(Clipboard*);
>+    explicit DragImageLoader(Clipboard*);
>     void startLoading(CachedResourceHandle<CachedImage>&);
>     void stopLoading(CachedResourceHandle<CachedImage>&);
> 
> private:
>-    DragImageLoader(Clipboard*);
>     virtual void imageChanged(CachedImage*, const IntRect*) override;
>     Clipboard* m_clipboard;
> };
>@@ -242,7 +241,7 @@ void Clipboard::setDragImage(Element* element, int x, int y)
>     m_dragImage = image;
>     if (m_dragImage) {
>         if (!m_dragImageLoader)
>-            m_dragImageLoader = DragImageLoader::create(this);
>+            m_dragImageLoader = std::make_unique<DragImageLoader>(this);
>         m_dragImageLoader->startLoading(m_dragImage);
>     }
> 
>@@ -286,11 +285,6 @@ DragImageRef Clipboard::createDragImage(IntPoint& location) const
> 
> #endif
> 
>-PassOwnPtr<DragImageLoader> DragImageLoader::create(Clipboard* clipboard)
>-{
>-    return adoptPtr(new DragImageLoader(clipboard));
>-}
>-
> DragImageLoader::DragImageLoader(Clipboard* clipboard)
>     : m_clipboard(clipboard)
> {
>diff --git a/Source/WebCore/dom/Clipboard.h b/Source/WebCore/dom/Clipboard.h
>index 5ca2700b6bff595eb0b68d7b4c7ea6d6d89c8e15..6eb34b55e3e70e62b649f5c766d1e9d187a5b440 100644
>--- a/Source/WebCore/dom/Clipboard.h
>+++ b/Source/WebCore/dom/Clipboard.h
>@@ -114,7 +114,7 @@ namespace WebCore {
>         IntPoint m_dragLocation;
>         CachedResourceHandle<CachedImage> m_dragImage;
>         RefPtr<Element> m_dragImageElement;
>-        OwnPtr<DragImageLoader> m_dragImageLoader;
>+        std::unique_ptr<DragImageLoader> m_dragImageLoader;
> #endif
>     };
>

Actions: View | Formatted Diff | Diff

Attachments on bug 129078: 224699 | 224816 | 224817