Get rid of custom code for clipboard.clearData()

Source/core/dom/Clipboard.cpp

Index: Source/core/dom/Clipboard.cpp
diff --git a/Source/core/dom/Clipboard.cpp b/Source/core/dom/Clipboard.cpp
index 0217c3ee9a0f13ad900da4dfbf8bd01f506ec8bd..9e15c900636b922e489f3d15df62e87f171fa651 100644
--- a/Source/core/dom/Clipboard.cpp
+++ b/Source/core/dom/Clipboard.cpp
@@ -155,15 +155,10 @@ void Clipboard::clearData(const String& type)
     if (!canWriteData())
         return;
 
-    m_dataObject->clearData(normalizeType(type));
-}
-
-void Clipboard::clearAllData()
-{
-    if (!canWriteData())
-        return;
-
-    m_dataObject->clearAll();
+    if (type.isNull())

[haraken, 2013/10/04 22:41:22]

To keep the current behavior, the if(type.isNull()) branch should come before
the if(!canWriteData()) branch.

[do-not-use, 2013/10/08 14:24:42]

Hmm, maybe I am misreading but I don't think so. The custom code was:
if (!args.Length()) {	
  clipboard->clearAllData();	
  return;	
}	
	
and Clipboard::clearAllData() was also calling if (!canWriteData()) first,
before calling m_dataObject->clearAll().

+        m_dataObject->clearAll();
+    else
+        m_dataObject->clearData(normalizeType(type));
 }
 
 String Clipboard::getData(const String& type) const