Add experimental Support for DOM3 KeyboardEvent key value

public/platform/Platform.h

Index: public/platform/Platform.h
diff --git a/public/platform/Platform.h b/public/platform/Platform.h
index 0f2d2bbfb334aa8f8fdb842a0fb158eb3bf0d8af..43bf102bc2db34d7660253a588a806d018fd5416 100644
--- a/public/platform/Platform.h
+++ b/public/platform/Platform.h
@@ -618,6 +618,18 @@ public:
     // Returns 0, if DOM code enum is not found.
     virtual int domEnumFromCodeString(const WebString& codeString) { return 0; }
 
+    // This method converts from the supplied DOM key enum to the

[Wez, 2015/03/04 23:00:26]

"... from the supplied embedder |key| enum to the corresponding DOM |key|
string."

[Habib Virji, 2015/03/09 15:14:27]

Done.

+    // embedder's DOM key string value for the key pressed. |domKey| values are
+    // based on the value defined in ui/events/keycodes/dom3/dom_key_data.h.

[Wez, 2015/03/04 23:00:25]

That header is a Chromium header - at the level if this interface there's no
need to specify details of the embedder (i.e. Chromium)'s values - they are
opaque to Blink.

[Habib Virji, 2015/03/09 15:14:27]

On 2015/03/04 23:00:25, Wez wrote:
In dom |code| CL, rbyers had suggested to specify where enum value are defined:
https://codereview.chromium.org/663523002/diff/220001/public/web/WebInputEven....
That's why included the path.

+    // Returns null string, if DOM key value is not found.

[Wez, 2015/03/04 23:00:26]

Do you mean returns null if the |domKey| value is not recognized? Or returns an
empty string?

[Habib Virji, 2015/03/09 15:14:27]

It is empty, corrected.

+    virtual WebString domKeyStringFromEnum(int domKey) { return WebString(); }
+
+    // This method converts from the suppled DOM Key value to the

[Wez, 2015/03/04 23:00:25]

nit: Capitalization of "Key" vs "key" in the previous comment.

[Habib Virji, 2015/03/09 15:14:27]

Done.

+    // embedder's DOM key enum for the key pressed. |keyString| is defined in
+    // ui/events/keycodes/dom3/dom_key_data.h.

[Wez, 2015/03/04 23:00:26]

See above; the returned domEnum values are opaque to Blink.

[Habib Virji, 2015/03/09 15:14:27]

Please suggest, rbyers did mention to include even if it is on chromium side. Is
it okay to keep it or should i remove it?

+    // Returns 0, if DOM key enum is not found.

[Wez, 2015/03/04 23:00:25]

nit: Remove the comma

[Habib Virji, 2015/03/09 15:14:27]

Done.

+    virtual int domEnumFromKeyString(const WebString& keyString) { return 0; }

[Wez, 2015/03/04 23:00:26]

domKeyEnumFromString

[Habib Virji, 2015/03/09 15:14:27]

Done.

+
     // Quota -----------------------------------------------------------
 
     // Queries the storage partition's storage usage and quota information.