Refactor WebCore::DragImage.

Source/core/platform/chromium/ClipboardChromium.cpp

 /*
  * Copyright (C) 2006, 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2008, 2009 Google Inc.
  *
  * 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.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 305 matching lines @@
 void ClipboardChromium::setDragImage(CachedImage* img, const IntPoint& loc)
 {
     setDragImage(img, 0, loc);
 }
 
 void ClipboardChromium::setDragImageElement(Node* node, const IntPoint& loc)
 {
     setDragImage(0, node, loc);
 }
 
-DragImageRef ClipboardChromium::createDragImage(IntPoint& loc) const
+PassOwnPtr<DragImage> ClipboardChromium::createDragImage(IntPoint& loc) const
 {
-    DragImageRef result = 0;
     if (m_dragImageElement) {
         if (m_frame) {
-            result = m_frame->nodeImage(m_dragImageElement.get());
             loc = m_dragLoc;
+            return m_frame->nodeImage(m_dragImageElement.get());
         }
     } else if (m_dragImage) {
-        result = createDragImageFromImage(m_dragImage->image());
         loc = m_dragLoc;
+        return DragImage::create(m_dragImage->image());
     }
-    return result;
+    return nullptr;
 }
 
 static CachedImage* getCachedImage(Element* element)
 {
     // Attempt to pull CachedImage from element
     ASSERT(element);
     RenderObject* renderer = element->renderer();
     if (!renderer || !renderer->isImage())
         return 0;
 
@@ skipping 112 matching lines @@
 
 PassRefPtr<DataTransferItemList> ClipboardChromium::items()
 {
     // FIXME: According to the spec, we are supposed to return the same collection of items each
     // time. We now return a wrapper that always wraps the *same* set of items, so JS shouldn't be
     // able to tell, but we probably still want to fix this.
     return DataTransferItemListPolicyWrapper::create(this, m_dataObject);
 }
 
 } // namespace WebCore