Summary: | REGRESSION (r181358): nicescroll.js main-frame scrolling override is broken | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||||
Component: | DOM | Assignee: | Tim Horton <thorton> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | andersca, ap, bdakin, darin, jimoase, jonlee, sam, simon.fraser, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Tim Horton
2015-03-19 18:03:38 PDT
Ouch! If this was closer to the time I made the change I would say "just roll out". We’ll need to reduce this to a smaller test case to see what caused it. I can’t work on that until tonight. Is there someone who wants to start tracking it down? The root cause is that ("ontouchstart" in window) is true now, and was false before. People are using that for feature detection... but it looks like the patch intentionally changed it? Also affects apple.com/watch *** Bug 143054 has been marked as a duplicate of this bug. *** Created attachment 249542 [details]
Patch
Comment on attachment 249542 [details]
Patch
I think that there are platform-specific results for at least one of those layout tests in both mavericks and win that you will need to update.
Created attachment 249544 [details]
Patch
Mass move bugs into the DOM component. |