Bug 230702 – api-ios tests passed (instead of crash or timeout) on buggy patch for Bug 230227

• Component: Tools / Tests
• Status: RESOLVED INVALID

[David Kilzer (:ddkilzer)]

api-ios tests in TestWebKitAPI passed (instead of crashing) on patch for Bug 230227.

The original commit for Bug 230227 had a bug where `m_weakObserver` was not set on non-macOS platforms (including Mac Catalyst and iOS), which should have caused crashes when running api-ios tests, but did not:

  Bug 230227:  <https://bugs.webkit.org/show_bug.cgi?id=230227>
  Original commit:  <https://commits.webkit.org/changeset/282632/webkit>
  Commit to fix the bug:  <https://trac.webkit.org/changeset/282849/webkit>
  Test run missing failures:  <https://ews-build.webkit.org/#/builders/9/builds/54899>

I'm still trying to reproduce the expected failure locally, but I'm filing this to track the issue.

[David Kilzer (:ddkilzer)]

Created attachment 439073 [details]
Back out the fix from r282849

This patch should cause crashes in api-ios tests.

Posting this to test that theory.

[Alexey Proskuryakov]

Were you able to reproduce locally?

[David Kilzer (:ddkilzer)]

(In reply to Alexey Proskuryakov from comment #2)
> Were you able to reproduce locally?

No, a ridiculous number of iossim tests timed out even with the bug fixed (on both Big Sur and Monterey), so I gave up trying locally.

[Tim Horton]

(On the other hand, I know of at least 3 people who could reproduce locally, including myself :P so SOMETHING interesting is happening here... but maybe it doesn't matter)