New issue
Advanced search Search tips

Issue 701381 link

Starred by 1 user
Status: Duplicate
Merged: issue 603614
Owner: ----
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Interface name argument of Document.createEvent() should be ASCII-case-insensitive

Project Member Reported by tkent@chromium.org, Mar 14 2017 
Chrome Version: 59 canary
OS: All but iOS

What steps will reproduce the problem?
(1) Open http://w3c-test.org/dom/nodes/Document-createEvent.html

What is the expected result?
No many FAILs

What happens instead?
102 FAILs

Please use labels and text to provide additional information.

https://dom.spec.whatwg.org/#dom-document-createevent
> If interface is an ASCII case-insensitive match for any of the strings in the first column in the following table, ...

Comment 1 by huy.duon...@gmail.com, Mar 14 2017 

Hi currently I'm working on this issue.
The problem is we creates the event without ignore case-sensitive 
https://cs.chromium.org/chromium/src/out/Debug/gen/blink/core/Event.cpp?type=cs&l=13
Because the above file is generated by make_event_factory.py https://cs.chromium.org/chromium/src/third_party/WebKit/Source/build/scripts/make_event_factory.py?q=make_event_fac&l=1
So basicly we need to change make_event_factory.py (add AnimationEvent, BeforeUnloadEvent... to create_event_whitelist)
Please let me know if this is correct.
Thanks.

Comment 2 by huy.duon...@gmail.com, Mar 18 2017 

Hi
This is my patch for this issue. Could you please review it and give me some advice.
Thanks and Best Regards.

https://codereview.chromium.org/2755313002/

Comment 3 by foolip@chromium.org, Mar 21 2017

Mergedinto: 603614
Status: Duplicate (was: Available)
I'm merging this into the older  issue 603614 , can you update the CL to point to that instead?

Comment 4 by huy.duon...@gmail.com, Mar 23 2017 

Hi foolip
I updated the CL to point to  issue 603614 .
►

Sign in to add a comment