194545 2019-02-12 11:02:55 -0800 Flaky API Test: TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack 2019-02-27 16:48:26 -0800 1 1 1 Unclassified WebKit Tools / Tests Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=194480 https://bugs.webkit.org/show_bug.cgi?id=194481 InRadar P2 Normal --- 1 aakash_jain cdumez aakash_jain achristensen ap beidson cdumez commit-queue ggaren koivisto lforschler rniwa webkit-bot-watchers-bugzilla webkit-bug-importer youennf oldest_to_newest 1505283 0 aakash_jain 2019-02-12 11:02:55 -0800 TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack seems flaky. In https://ews-build.webkit-uat.org/#/builders/20/builds/889, the test passed in run-api-tests step. However, in the immediately next retry step (re-run-api-tests), it failed. 1510425 1 aakash_jain 2019-02-27 05:55:11 -0800 Can we prioritize this? I am working on EWS for API tests, and this flaky failure is creating problem. 1510497 2 cdumez 2019-02-27 10:52:40 -0800 TestWebKitAPI.ProcessSwap.SuspendedPageLimit Received data during response processing, queuing it. Received data during response processing, queuing it. Received data during response processing, queuing it. Received data during response processing, queuing it. Received data during response processing, queuing it. /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:2720 Expected equality of these values: [processPool _webProcessCountIgnoringPrewarmed] Which is: 4 (1U + maximumSuspendedPageCount) Which is: 3 1510498 3 cdumez 2019-02-27 10:54:03 -0800 It is actually: TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack Received data during response processing, queuing it. Received data during response processing, queuing it. /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484 Expected equality of these values: 3 numberOfDecidePolicyCalls Which is: 4 1510666 4 cdumez 2019-02-27 15:55:03 -0800 (In reply to Chris Dumez from comment #3) > It is actually: > TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack > Received data during response processing, queuing it. > Received data during response processing, queuing it. > > > /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/ > Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484 > Expected equality of these values: > 3 > numberOfDecidePolicyCalls > Which is: 4 Not sure how we can end up with an extra policy decision here... 1510668 5 cdumez 2019-02-27 15:58:49 -0800 (In reply to Chris Dumez from comment #4) > (In reply to Chris Dumez from comment #3) > > It is actually: > > TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack > > Received data during response processing, queuing it. > > Received data during response processing, queuing it. > > > > > > /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/ > > Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484 > > Expected equality of these values: > > 3 > > numberOfDecidePolicyCalls > > Which is: 4 > > Not sure how we can end up with an extra policy decision here... Oh, I have an idea, I'll confirm. 1510676 6 363151 cdumez 2019-02-27 16:13:41 -0800 Created attachment 363151 Patch 1510693 7 363151 commit-queue 2019-02-27 16:43:43 -0800 Comment on attachment 363151 Patch Clearing flags on attachment: 363151 Committed r242179: <https://trac.webkit.org/changeset/242179> 1510694 8 commit-queue 2019-02-27 16:43:45 -0800 All reviewed patches have been landed. Closing bug. 1510699 9 webkit-bug-importer 2019-02-27 16:48:26 -0800 <rdar://problem/48456644> 363151 2019-02-27 16:13:41 -0800 2019-02-27 16:43:43 -0800 Patch bug-194545-20190227161339.patch text/plain 1704 cdumez U3VidmVyc2lvbiBSZXZpc2lvbjogMjQyMTI3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggYWVkMWRkZjBlNDA1Y2NmY2UwYWQzOGQxOGU4ZTI1MzEz YTk3YmM5YS4uOTkwZTUwNTk1NTQ3MWMzZDU5ZTYwZDAyMGFmNWMxMzRkOWVlOWIwZCAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1 IEBACisyMDE5LTAyLTI3ICBDaHJpcyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CisKKyAgICAg ICAgRmxha3kgQVBJIFRlc3Q6IFRlc3RXZWJLaXRBUEkuUHJvY2Vzc1N3YXAuTmF2aWdhdGVUb0Rh dGFVUkxUaGVuQmFjaworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j Z2k/aWQ9MTk0NTQ1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgTWFrZSBzdXJlIHRoZSB0ZXN0IG5hdmlnYXRlcyBmb3J3YXJkIGFuZCB0aGVuIGJhY2sg b25seSBvbmNlLiBQcmV2aW91c2x5LCBuYXZpZ2F0aW5nCisgICAgICAgIGJhY2sgd291bGQgdHJp Z2dlciBhIG5hdmlnYXRpb24gYWdhaW4gaW4gYSB0aW1lci4KKworICAgICAgICAqIFRlc3RXZWJL aXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvUHJvY2Vzc1N3YXBPbk5hdmlnYXRpb24ubW06CisKIDIw MTktMDItMjYgIE1hcmsgTGFtICA8bWFyay5sYW1AYXBwbGUuY29tPgogCiAgICAgICAgIFJlbW92 ZSByZW1haW5pbmcgcG9pc29uaW5nIGNvZGUuCmRpZmYgLS1naXQgYS9Ub29scy9UZXN0V2ViS2l0 QVBJL1Rlc3RzL1dlYktpdENvY29hL1Byb2Nlc3NTd2FwT25OYXZpZ2F0aW9uLm1tIGIvVG9vbHMv VGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9Qcm9jZXNzU3dhcE9uTmF2aWdhdGlvbi5t bQppbmRleCA5ZjE4MjgyOTQ1ZjEwMmE1MmMwOTA0MWIyNjVlZWU5MzE4ZWMzZjk0Li45ZTlhYjhl NmIwNDY1ZTc3YThjZTMwYTg1NTNlNTE3MzZmNmVlYTg1IDEwMDY0NAotLS0gYS9Ub29scy9UZXN0 V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL1Byb2Nlc3NTd2FwT25OYXZpZ2F0aW9uLm1tCisr KyBiL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvUHJvY2Vzc1N3YXBPbk5h dmlnYXRpb24ubW0KQEAgLTM3NDgsNiArMzc0OCwxMiBAQCBURVNUKFByb2Nlc3NTd2FwLCBOYXZp Z2F0ZVRvSW52YWxpZFVSTCkKIHN0YXRpYyBjb25zdCBjaGFyKiBuYXZpZ2F0ZVRvRGF0YVVSTFRo ZW5CYWNrQnl0ZXMgPSBSIlBTT05SRVNPVVJDRSgKIDxzY3JpcHQ+CiBvbnBhZ2VzaG93ID0gZnVu Y3Rpb24oZXZlbnQpIHsKKyAgICBpZiAoc2Vzc2lvblN0b3JhZ2UuZ2V0SXRlbSgnbmF2aWdhdGVk JykgPT0gJ3RydWUnKSB7CisgICAgICAgIHNlc3Npb25TdG9yYWdlLmNsZWFyKCk7CisgICAgICAg IHJldHVybjsKKyAgICB9CisgICAgc2Vzc2lvblN0b3JhZ2Uuc2V0SXRlbSgnbmF2aWdhdGVkJywg J3RydWUnKTsKKwogICAgIC8vIExvY2F0aW9uIGNoYW5nZXMgbmVlZCB0byBoYXBwZW4gb3V0c2lk ZSB0aGUgb25sb2FkIGhhbmRsZXIgdG8gZ2VuZXJhdGUgaGlzdG9yeSBlbnRyaWVzLgogICAgIHNl dFRpbWVvdXQoZnVuY3Rpb24oKSB7CiAgICAgICB3aW5kb3cubG9jYXRpb24uaHJlZiA9ICJkYXRh OnRleHQvaHRtbCw8Ym9keSBvbmxvYWQ9J2hpc3RvcnkuYmFjaygpJz48L2JvZHk+IjsK