queryCommandSupported() should return 'true' for cut/copy/paste editing commands

Source/core/editing/Editor.h

Index: Source/core/editing/Editor.h
diff --git a/Source/core/editing/Editor.h b/Source/core/editing/Editor.h
index f3b0e856a0b69830cb25e45ff78ac562b547b03b..21c433b5b1d030183869306d85bcc196753fd944 100644
--- a/Source/core/editing/Editor.h
+++ b/Source/core/editing/Editor.h
@@ -129,7 +129,7 @@ public:
         bool execute(const String& parameter = String(), Event* triggeringEvent = nullptr) const;
         bool execute(Event* triggeringEvent) const;
 
-        bool isSupported() const;
+        bool isSupported(bool enabled = false) const;

[dcheng, 2015/04/20 17:09:03]

I don't really understand why we added this default arg. It seems wrong.

[Abhijeet Kandalkar Slow, 2015/04/21 14:14:33]

current implementation of isSupported() check whether javaScript can access
clipboard. As it is not allowed it is returning false. 
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

isSupported() should only check whether command is present in CommandMap. If
command is supported, isEnabled() should check whether is it actually available
to use.

Default flag is added to identify whether isSupported() is invoked by
isEnabled(). If it is invoked by isEnabled() then only check whether javaScript
can access clipboard.

         bool isEnabled(Event* triggeringEvent = nullptr) const;
 
         TriState state(Event* triggeringEvent = nullptr) const;