RESOLVED FIXED 229200
REGRESSION (iOS 15): DoubleDown Casino app won't load past launch page
https://bugs.webkit.org/show_bug.cgi?id=229200
Summary REGRESSION (iOS 15): DoubleDown Casino app won't load past launch page
Wenson Hsieh
Reported 2021-08-17 12:45:44 PDT
Attachments
For EWS (4.32 KB, patch)
2021-08-17 13:45 PDT, Wenson Hsieh
no flags
Wenson Hsieh
Comment 1 2021-08-17 13:45:29 PDT
Myles C. Maxfield
Comment 2 2021-08-17 16:33:09 PDT
Comment on attachment 435712 [details] For EWS View in context: https://bugs.webkit.org/attachment.cgi?id=435712&action=review > Source/WebKit/ChangeLog:43 > + Unfortunately, I have not been able to come up with a layout test that reliably reproduces this scenario (yet). Ideally a new test would be good, but I think it's evident that this is already tested by existing tests, which are currently flakey now.
Wenson Hsieh
Comment 3 2021-08-17 16:37:01 PDT
Comment on attachment 435712 [details] For EWS View in context: https://bugs.webkit.org/attachment.cgi?id=435712&action=review Thanks for the review! (Let's wait for ios-wk2 before adding the cq+) >> Source/WebKit/ChangeLog:43 >> + Unfortunately, I have not been able to come up with a layout test that reliably reproduces this scenario (yet). > > Ideally a new test would be good, but I think it's evident that this is already tested by existing tests, which are currently flakey now. Indeed — I'm still actively working on a layout test for this. I suspect that figuring out how to trigger the race conditions that cause this bug will also help us write more tests in this area, moving forward.
Wenson Hsieh
Comment 4 2021-08-17 19:46:01 PDT
(In reply to Wenson Hsieh from comment #3) > Comment on attachment 435712 [details] > For EWS > > View in context: > https://bugs.webkit.org/attachment.cgi?id=435712&action=review > > Thanks for the review! > > (Let's wait for ios-wk2 before adding the cq+) > > >> Source/WebKit/ChangeLog:43 > >> + Unfortunately, I have not been able to come up with a layout test that reliably reproduces this scenario (yet). > > > > Ideally a new test would be good, but I think it's evident that this is already tested by existing tests, which are currently flakey now. > > Indeed — I'm still actively working on a layout test for this. > I suspect that figuring out how to trigger the race conditions that cause > this bug will also help us write more tests in this area, moving forward. (Tracking a test in https://bugs.webkit.org/show_bug.cgi?id=229222)
Wenson Hsieh
Comment 5 2021-08-18 06:32:33 PDT
Comment on attachment 435712 [details] For EWS Looks like iOS test failures are unrelated.
EWS
Comment 6 2021-08-18 06:42:55 PDT
Committed r281186 (240631@main): <https://commits.webkit.org/240631@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 435712 [details].
Note You need to log in before you can comment on or make changes to this bug.