238581 2022-03-30 16:49:44 -0700 REGRESSION(r292107): [ Mac iOS ] http/tests/paymentrequest/updateWith-shippingOptions.https.html is a constant text failure 2022-07-21 16:19:09 -0700 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=238306 https://bugs.webkit.org/show_bug.cgi?id=243083 InRadar P2 Normal --- 1 Morningstar hi cdumez hi webkit-bot-watchers-bugzilla webkit-bug-importer zalan oldest_to_newest 1856613 0 Morningstar 2022-03-30 16:49:44 -0700 http/tests/paymentrequest/updateWith-shippingOptions.https.html Is a constant text failure on all all queues. regression appears to be r292107, but its notable that r291811 was also a regression for this test; causing a constant failure on Monterey specifically. HISTORY: https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fpaymentrequest%2FupdateWith-shippingOptions.https.html DIFF: --- /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/http/tests/webAPIStatistics/canvas-read-and-write-data-collection-expected.txt +++ /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/http/tests/webAPIStatistics/canvas-read-and-write-data-collection-actual.txt @@ -9,3 +9,11 @@ isPrevalentResource: No isVeryPrevalentResource: No dataRecordsRemoved: 0 +Registrable domain: nullOrigin + hadUserInteraction: No + mostRecentUserInteraction: -1 + grandfathered: No + IsScheduledForAllButCookieDataRemoval: No + isPrevalentResource: No + isVeryPrevalentResource: No + dataRecordsRemoved: 0 DIFF-URL: https://build.webkit.org/results/Apple-BigSur-Release-WK2-Tests/r292107%20(8585)/http/tests/webAPIStatistics/canvas-read-and-write-data-collection-diff.txt 1856614 1 webkit-bug-importer 2022-03-30 16:51:05 -0700 <rdar://problem/91074265> 1856630 2 zalan 2022-03-30 18:10:32 -0700 I am puzzled by this regression point and if it truly regressed at r291811, then the original author should change the test so that it's not sensitive to a text measuring change like this. 1856882 3 hi 2022-03-31 11:23:39 -0700 yeah im not sure how this relates to r291811 at all i think r292107 is really the cause 1856884 4 456259 hi 2022-03-31 11:27:14 -0700 Created attachment 456259 [fast-cq] Patch 1856890 5 Morningstar 2022-03-31 11:29:55 -0700 Noticed this DIFF is incorrect. Must have clicked an incorrect link. Correct DIFF: --- /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/http/tests/paymentrequest/updateWith-shippingOptions.https-expected.txt +++ /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/http/tests/paymentrequest/updateWith-shippingOptions.https-actual.txt @@ -1,5 +1,5 @@ CONSOLE MESSAGE: WebKit currently uses the first shipping option even if other shipping options are marked as selected. PASS Calling `updateWith` with a new `shippingOptions` without `requestShipping` should not update any values. -FAIL Calling `updateWith` with a new `shippingOptions` should not update any other values. assert_equals: selected shipping option should change expected "shippingId4" but got "shippingId3" +FAIL Calling `updateWith` with a new `shippingOptions` should not update any other values. assert_equals: shipping option 0 selected should change expected (boolean) false but got (undefined) undefined Correct DIFF URL: https://build.webkit.org/results/Apple-BigSur-Release-WK2-Tests/r292156%20(8603)/http/tests/paymentrequest/updateWith-shippingOptions.https-diff.txt 1856891 6 Morningstar 2022-03-31 11:31:29 -0700 (In reply to Devin Rousso from comment #3) > yeah im not sure how this relates to r291811 at all > > i think r292107 is really the cause I think r292107 is the real cause as well, but I did notice looking at the history, a strange failure after r291811. 1856963 7 456259 cdumez 2022-03-31 14:03:21 -0700 Comment on attachment 456259 [fast-cq] Patch Clearing flags on attachment: 456259 Committed r292173 (249080@trunk): <https://commits.webkit.org/249080@trunk> 1856964 8 cdumez 2022-03-31 14:03:23 -0700 All reviewed patches have been landed. Closing bug. 1859097 9 Morningstar 2022-04-06 17:15:44 -0700 *** Bug 238908 has been marked as a duplicate of this bug. *** 456259 2022-03-31 11:27:14 -0700 2022-03-31 14:03:21 -0700 [fast-cq] Patch 238581.diff text/plain 1791 hi ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCAyMjg5Njk1NTFmNzUuLjUyNTY0YTNkNDZkNSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVz dHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBA CisyMDIyLTAzLTMxICBEZXZpbiBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAg ICBSRUdSRVNTSU9OKHIyOTIxMDcpOiBbIE1hYyBpT1MgXSBodHRwL3Rlc3RzL3BheW1lbnRyZXF1 ZXN0L3VwZGF0ZVdpdGgtc2hpcHBpbmdPcHRpb25zLmh0dHBzLmh0bWwgaXMgYSBjb25zdGFudCB0 ZXh0IGZhaWx1cmUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTIzODU4MQorICAgICAgICA8cmRhcjovL3Byb2JsZW0vOTEwNzQyNjU+CisKKyAgICAgICAg VW5yZXZpZXdlZCwgYWRqdXN0IHRlc3QgcmVzdWx0cy4gVGhpcyB0ZXN0IGlzIGV4cGVjdGVkIHRv IGZhaWwgcHJlIG1hY09TIDEyLjMgYW5kIGlPUyAxNS40LgorCisgICAgICAgICogaHR0cC90ZXN0 cy9wYXltZW50cmVxdWVzdC91cGRhdGVXaXRoLXNoaXBwaW5nT3B0aW9ucy5odHRwcy1leHBlY3Rl ZC50eHQ6CisKIDIwMjItMDMtMzAgIERldmluIFJvdXNzbyAgPGRyb3Vzc29AYXBwbGUuY29tPgog CiAgICAgICAgIFVucmV2aWV3ZWQsIGZpeCB0ZXN0IGFmdGVyIHIyODU1MjEgYW5kIHIyNzE3MzUK ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvcGF5bWVudHJlcXVlc3QvdXBkYXRl V2l0aC1zaGlwcGluZ09wdGlvbnMuaHR0cHMtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvaHR0 cC90ZXN0cy9wYXltZW50cmVxdWVzdC91cGRhdGVXaXRoLXNoaXBwaW5nT3B0aW9ucy5odHRwcy1l eHBlY3RlZC50eHQKaW5kZXggOTg1Zjg3Y2U0MjQzLi5iNmNlODg1NWUwYTQgMTAwNjQ0Ci0tLSBh L0xheW91dFRlc3RzL2h0dHAvdGVzdHMvcGF5bWVudHJlcXVlc3QvdXBkYXRlV2l0aC1zaGlwcGlu Z09wdGlvbnMuaHR0cHMtZXhwZWN0ZWQudHh0CisrKyBiL0xheW91dFRlc3RzL2h0dHAvdGVzdHMv cGF5bWVudHJlcXVlc3QvdXBkYXRlV2l0aC1zaGlwcGluZ09wdGlvbnMuaHR0cHMtZXhwZWN0ZWQu dHh0CkBAIC0xLDUgKzEsNSBAQAogQ09OU09MRSBNRVNTQUdFOiBXZWJLaXQgY3VycmVudGx5IHVz ZXMgdGhlIGZpcnN0IHNoaXBwaW5nIG9wdGlvbiBldmVuIGlmIG90aGVyIHNoaXBwaW5nIG9wdGlv bnMgYXJlIG1hcmtlZCBhcyBzZWxlY3RlZC4KIAogUEFTUyBDYWxsaW5nIGB1cGRhdGVXaXRoYCB3 aXRoIGEgbmV3IGBzaGlwcGluZ09wdGlvbnNgIHdpdGhvdXQgYHJlcXVlc3RTaGlwcGluZ2Agc2hv dWxkIG5vdCB1cGRhdGUgYW55IHZhbHVlcy4KLUZBSUwgQ2FsbGluZyBgdXBkYXRlV2l0aGAgd2l0 aCBhIG5ldyBgc2hpcHBpbmdPcHRpb25zYCBzaG91bGQgbm90IHVwZGF0ZSBhbnkgb3RoZXIgdmFs dWVzLiBhc3NlcnRfZXF1YWxzOiBzZWxlY3RlZCBzaGlwcGluZyBvcHRpb24gc2hvdWxkIGNoYW5n ZSBleHBlY3RlZCAic2hpcHBpbmdJZDQiIGJ1dCBnb3QgInNoaXBwaW5nSWQzIgorRkFJTCBDYWxs aW5nIGB1cGRhdGVXaXRoYCB3aXRoIGEgbmV3IGBzaGlwcGluZ09wdGlvbnNgIHNob3VsZCBub3Qg dXBkYXRlIGFueSBvdGhlciB2YWx1ZXMuIGFzc2VydF9lcXVhbHM6IHNoaXBwaW5nIG9wdGlvbiAw IHNlbGVjdGVkIHNob3VsZCBjaGFuZ2UgZXhwZWN0ZWQgKGJvb2xlYW4pIGZhbHNlIGJ1dCBnb3Qg KHVuZGVmaW5lZCkgdW5kZWZpbmVkCiAK