Bug 80491 - [EFL] Key press event is not processed properly.
Summary: [EFL] Key press event is not processed properly.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-06 22:29 PST by ChangSeok Oh
Modified: 2012-03-07 09:15 PST (History)
4 users (show)

See Also:


Attachments
Patch (6.02 KB, patch)
2012-03-07 02:58 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff
Patch (4.58 KB, patch)
2012-03-07 06:43 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ChangSeok Oh 2012-03-06 22:29:30 PST
This issue is related with mutation-observer feature.
If I press enter key, then a keyboard event for "Enter key" should be processed and reach to the mutation observer, but it doesn't.
Comment 1 ChangSeok Oh 2012-03-07 02:58:20 PST
Created attachment 130583 [details]
Patch
Comment 2 Gyuyoung Kim 2012-03-07 04:56:46 PST
Comment on attachment 130583 [details]
Patch

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

It looks that you modify nits of EflKeyboardUtilities | PlatformKeyboardEventEfl as well as add singleCharacterString(). I think this is not match for ChangeLog. I think it is better to split this patch into two patches.

> Source/WebCore/ChangeLog:17
> +        fast/mutation/end-of-task-delivery.html

Trivial nit : Missing "Test:" in front of test case.
Comment 3 ChangSeok Oh 2012-03-07 06:43:24 PST
Created attachment 130615 [details]
Patch
Comment 4 ChangSeok Oh 2012-03-07 06:51:07 PST
Comment on attachment 130583 [details]
Patch

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

Thanks for comment. I'll deal with remained changes in bug80511.

>> Source/WebCore/ChangeLog:17
>> +        fast/mutation/end-of-task-delivery.html
> 
> Trivial nit : Missing "Test:" in front of test case.

Done.
Comment 5 Gustavo Noronha (kov) 2012-03-07 07:26:58 PST
Comment on attachment 130615 [details]
Patch

OK, looks sensible
Comment 6 WebKit Review Bot 2012-03-07 09:15:22 PST
Comment on attachment 130615 [details]
Patch

Clearing flags on attachment: 130615

Committed r110061: <http://trac.webkit.org/changeset/110061>
Comment 7 WebKit Review Bot 2012-03-07 09:15:27 PST
All reviewed patches have been landed.  Closing bug.