Add experimental Support for DOM3 KeyboardEvent key value

public/platform/Platform.h

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 620 matching lines @@
     // based on the value defined in ui/events/keycodes/dom4/keycode_converter_data.h.
     // Returns null string, if DOM code value is not found.
     virtual WebString domCodeStringFromEnum(int domCode) { return WebString(); }
 
     // This method converts from the suppled DOM code value to the
     // embedder's DOM code enum for the key pressed. |codeString| is defined in
     // ui/events/keycodes/dom4/keycode_converter_data.h.
     // 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/05/06 01:32:13]

Suggest: "... from the embedder-supplied |domKey| enum value to ..."

+    // corresponding DOM |key| string value for the key pressed. |domKey| values are

[Wez, 2015/05/06 01:32:13]

nit: Remove "for the key pressed" - the call accepts just a string containing a
DOM UI Events |key| meaning; it doesn't contain anything specific to a key
press, release, etc.

Also suggest wrapping consistently to 80 chars, since that's what you've done
for the rest of the comment. :)

+    // based on the value defined in ui/events/keycodes/dom3/dom_key_data.h.

[Wez, 2015/05/06 01:32:13]

Reword: "The DOM |key| enum <-> string mappings for Chromium are defined in ..."

or, better still, just remove the reference to the Chromium path, since that's a
Chromium- (and in general an embedder-) specific detail.

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

[Wez, 2015/05/06 01:32:13]

nit: No need for comma between "string" and "if". Add "the" between "if" and
"DOM", and between "Returns" and "empty".

+    virtual WebString domKeyStringFromEnum(int domKey) { return WebString(); }
+
+    // This method converts from the suppled DOM |key| value to the
+    // embedder's DOM |key| enum for the key pressed. |keyString| is defined in

[Wez, 2015/05/06 01:32:13]

Suggest: "... to the embedder's corresponding enum value."

+    // ui/events/keycodes/dom3/dom_key_data.h.
+    // Returns 0 if DOM key enum is not found.

[Wez, 2015/05/06 01:32:13]

nit: Suggest "Returns 0 if |keyString| is not recognized."

+    virtual int domKeyEnumFromString(const WebString& keyString) { return 0; }
+
     // Quota -----------------------------------------------------------
 
     // Queries the storage partition's storage usage and quota information.
     // WebStorageQuotaCallbacks::didQueryStorageUsageAndQuota will be called
     // with the current usage and quota information for the partition. When
     // an error occurs WebStorageQuotaCallbacks::didFail is called with an
     // error code.
     virtual void queryStorageUsageAndQuota(
         const WebURL& storagePartition,
         WebStorageQuotaType,
@@ skipping 42 matching lines @@
 protected:
     BLINK_PLATFORM_EXPORT Platform();
     virtual ~Platform() { }
 
     WebThread* m_mainThread;
 };
 
 } // namespace blink
 
 #endif