Bug 26878 - Incorrect event.keyIdentifier value for F10 and F11 keys
Summary: Incorrect event.keyIdentifier value for F10 and F11 keys
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2009-06-30 23:55 PDT by Yury Semikhatsky
Modified: 2022-10-06 13:26 PDT (History)
4 users (show)

See Also:


Attachments
patch (1.02 KB, patch)
2011-12-28 19:29 PST, rahul abrol
darin: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2009-06-30 23:55:54 PDT
Steps to reproduce:
1. Load the following HTML Chrome or Safari on Windows.

<html>
<body onkeydown="alert('event.keyIdentifier = ' + event.keyIdentifier);">
</body>
</html>

2. Press F10 and F11 keys

Result:
For F10 key you'll see event.keyIdentifier = F11 alert
For F11 key it will be event.keyIdentifier = U+005A
Comment 1 rahul abrol 2011-12-28 19:29:56 PST
Created attachment 120715 [details]
patch

added missing f10 and f11 entries.
Comment 2 WebKit Review Bot 2011-12-28 19:31:26 PST
Attachment 120715 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files']" exit_code: 1

Total errors found: 0 in 0 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Darin Adler 2011-12-28 19:45:52 PST
Comment on attachment 120715 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=120715&action=review

> b/Source/WebCore/ChangeLog:8
> +        No new tests required.

Why not?
Comment 4 rahul abrol 2011-12-28 22:38:44 PST
(In reply to comment #3)
> (From update of attachment 120715 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=120715&action=review
> 
> > b/Source/WebCore/ChangeLog:8
> > +        No new tests required.
> 
> Why not?

i don't have the source tree on this machine :-\
Comment 5 Darin Adler 2013-05-08 09:11:16 PDT
Comment on attachment 120715 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=120715&action=review

> b/Source/WebCore/ChangeLog:1
> +2011-12-28 Rahul Abrol <solushex@gmail.com>

This line is formatted incorrectly.

>>> b/Source/WebCore/ChangeLog:8
>>> +        No new tests required.
>> 
>> Why not?
> 
> i don't have the source tree on this machine :-\

So it’s wrong to say “no new tests required”. The change log would be better without this comment.
Comment 6 WebKit Commit Bot 2013-05-08 09:38:01 PDT
Comment on attachment 120715 [details]
patch

Rejecting attachment 120715 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 120715, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
    -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 149741 = 15972cccde39a3cae0119ab06d4b009fd3a1b76b
r149742 = 3ee31df2f3c5f56992db99f868a87f2b746f11b5
r149743 = e69d6637a7718f1baa7118e3fc51122fc1b0d5e8
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.appspot.com/results/408428
Comment 7 EWS 2022-10-06 13:25:45 PDT
Committed 255238@main (25a848ab9af4): <https://commits.webkit.org/255238@main>

Reviewed commits have been landed. Closing PR #5054 and removing active labels.
Comment 8 Radar WebKit Bug Importer 2022-10-06 13:26:25 PDT
<rdar://problem/100867561>