Summary: | REGRESSION (r286346): ResourceLoadDelegate.LoadInfo fails | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||
Component: | New Bugs | Assignee: | Tim Horton <thorton> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, darin, simon.fraser, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Tim Horton
2021-11-30 23:58:13 PST
Created attachment 445533 [details]
Patch
Committed r286368 (244727@main): <https://commits.webkit.org/244727@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 445533 [details]. Comment on attachment 445533 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445533&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/ResourceLoadDelegate.mm:-382 > - EXPECT_EQ(archiveData.length, 607ull); Another comment from my new crusade, on this deleted code, so that observers will learn what I have recently learned: This is backwards, the actual value first and then the expected. The GTest macros take the expected value first and the actual value second. Getting this right helps the clarity of the message when the expectation is not met. (In reply to Darin Adler from comment #4) > Another comment from my new crusade, on this deleted code, so that observers > will learn what I have recently learned: This is backwards, the actual value > first and then the expected. The GTest macros take the expected value first > and the actual value second. Getting this right helps the clarity of the > message when the expectation is not met. Now I am not sure; the error message someone mentioned in a Slack discussion seems to be more even-handed than I thought, so my comment may be out of date. |