Bug 261371
| Summary: | Upgrade stepUp()/stepDown() implementation to match spec | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Forms | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | cdumez, karlcow, webkit-bug-importer, wenson_hsieh |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
This is more for merge of following Blink commit:
https://src.chromium.org/viewvc/blink?view=revision&revision=166525
I am separating it to not over-complicate following bug 254761: https://github.com/WebKit/WebKit/pull/17624
Just wanted to raise so we can fix it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Karl Dubost
seeAlso https://bugs.webkit.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&list_id=9536592&query_format=advanced&short_desc=stepUp%20&short_desc_type=allwordssubstr
Radar WebKit Bug Importer
<rdar://problem/115595585>
Ahmad Saleem
It has pre-requisite - https://src.chromium.org/viewvc/blink?view=revision&revision=163656
__
I am merging few commits (three in total) with PR - https://github.com/WebKit/WebKit/pull/22889
and then I can merge 'pre-requisite' and this.
Later, I have to add 'NaN' handling to match others and then also do 'applyStep' alignment by merging few more Blink patches and then we might be OK and synced with Blink. :-)
Ahmad Saleem
Ended up doing ToT version in PR - https://github.com/WebKit/WebKit/pull/22889
So this bug is not needed.