RESOLVED FIXED206383
Fix assertion when test fast/events/ios/rotation/basic-rotation.html is followed by fast/events/ios/rotation/do-not-shrink-to-fit-content-after-rotation.html 
https://bugs.webkit.org/show_bug.cgi?id=206383
Summary Fix assertion when test fast/events/ios/rotation/basic-rotation.html is follo...
Megan Gardner
Reported 2020-01-16 16:10:21 PST
Early out of size transition if no size change
Attachments
Patch (1.84 KB, patch)
2020-01-16 16:13 PST, Megan Gardner 		no flags
DetailsFormatted DiffDiff
Patch (2.08 KB, patch)
2020-01-17 10:15 PST, Megan Gardner 		no flags
DetailsFormatted DiffDiff
Patch for landing (2.86 KB, patch)
2020-01-17 13:10 PST, Megan Gardner 		no flags
DetailsFormatted DiffDiff
Patch for landing (2.68 KB, patch)
2020-01-17 13:13 PST, Megan Gardner 		no flags
DetailsFormatted DiffDiff
Patch for landing (1.90 KB, patch)
2020-01-17 13:14 PST, Megan Gardner 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Megan Gardner
Comment 1 2020-01-16 16:13:22 PST
Created attachment 387977 [details] Patch
Megan Gardner
Comment 2 2020-01-16 16:14:03 PST
<rdar://problem/58618513>
Tim Horton
Comment 3 2020-01-16 16:20:36 PST
Comment on attachment 387977 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387977&action=review > Tools/ChangeLog:3 > + Early out of size transition if no size change The title needs to be a liiiitle more descriptive (I should at least know this is about WKTR) > Tools/ChangeLog:9 > + Rotation tests are asserting because _didEndRotation is being called multiple times. What changed‽ Surely not this code.
Simon Fraser (smfr)
Comment 4 2020-01-16 16:25:40 PST
Comment on attachment 387977 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387977&action=review >> Tools/ChangeLog:3 >> + Early out of size transition if no size change > > The title needs to be a liiiitle more descriptive (I should at least know this is about WKTR) Yeah it should be "fix assertion when test foo runs after test bar (or at least mention the specific steps to reproduce in the bugzilla).
Megan Gardner
Comment 5 2020-01-17 10:15:45 PST
Created attachment 388054 [details] Patch
Tim Horton
Comment 6 2020-01-17 10:19:01 PST
Comment on attachment 388054 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388054&action=review > Tools/ChangeLog:11 > + size that the view already is. Thereform, iof no size is changing, early out, iof
Wenson Hsieh
Comment 7 2020-01-17 10:19:46 PST
(In reply to Tim Horton from comment #6) > Comment on attachment 388054 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=388054&action=review > > > Tools/ChangeLog:11 > > + size that the view already is. Thereform, iof no size is changing, early out, > > iof (Nit - Thereform too)
Simon Fraser (smfr)
Comment 8 2020-01-17 11:05:18 PST
Comment on attachment 388054 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388054&action=review > Tools/WebKitTestRunner/ios/PlatformWebViewIOS.mm:133 > + Whitespace!
Megan Gardner
Comment 9 2020-01-17 13:10:11 PST
Created attachment 388079 [details] Patch for landing
Megan Gardner
Comment 10 2020-01-17 13:13:28 PST
Created attachment 388080 [details] Patch for landing
Megan Gardner
Comment 11 2020-01-17 13:14:30 PST
Created attachment 388081 [details] Patch for landing
WebKit Commit Bot
Comment 12 2020-01-17 13:57:35 PST
Comment on attachment 388081 [details] Patch for landing Clearing flags on attachment: 388081 Committed r254765: <https://trac.webkit.org/changeset/254765>
WebKit Commit Bot
Comment 13 2020-01-17 13:57:36 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.