Bug 224085 - REGRESSION (r274460?): Infinite recursion on fullscreen/full-screen-remove-ancestor.html in WTR::InjectedBundlePage::closeFullScreen
Summary: REGRESSION (r274460?): Infinite recursion on fullscreen/full-screen-remove-an...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Hutchison
URL:
Keywords: InRadar
: 227457 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-04-01 15:49 PDT by Ryan Haddad
Modified: 2021-08-05 15:18 PDT (History)
7 users (show)

See Also:


Attachments
crash log (166.88 KB, text/plain)
2021-04-01 15:49 PDT, Ryan Haddad
no flags Details
Updated Test Expectations (1.65 KB, patch)
2021-07-07 10:25 PDT, Eric Hutchison
no flags Details | Formatted Diff | Diff
Patch (4.38 KB, patch)
2021-07-29 21:27 PDT, Jean-Yves Avenard [:jya]
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2021-04-01 15:49:29 PDT
fullscreen/full-screen-remove-ancestor.html appears to have become a flaky crash on macOS bots. It seems that this may have started after https://trac.webkit.org/changeset/274460/webkit landed

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   libsystem_trace.dylib         	0x00007fff200b4ac5 0x7fff200ad000 + 31429
1   libsystem_trace.dylib         	0x00007fff200b47d2 0x7fff200ad000 + 30674
2   libsystem_trace.dylib         	0x00007fff200b4727 0x7fff200ad000 + 30503
3   libsystem_trace.dylib         	0x00007fff200b2a4e 0x7fff200ad000 + 23118
4   libsystem_trace.dylib         	0x00007fff200b2054 0x7fff200ad000 + 20564
5   libsystem_trace.dylib         	0x00007fff200b1fa9 0x7fff200ad000 + 20393
6   com.apple.WebCore             	0x000000010a85ba00 void WTF::Logger::log<WTF::Logger::LogSiteIdentifier, char [41]>(WTFLogChannel&, WTFLogLevel, WTF::Logger::LogSiteIdentifier const&, char const (&) [41]) + 256 (Logger.h:316)
7   com.apple.WebCore             	0x000000010b7f84a8 error<WTF::Logger::LogSiteIdentifier, char [41]> + 34 (Logger.h:154) [inlined]
8   com.apple.WebCore             	0x000000010b7f84a8 WebCore::FullscreenManager::willExitFullscreen() + 440 (FullscreenManager.cpp:465)
9   com.apple.WebKit              	0x0000000108932492 WebKit::WebFullScreenManager::willExitFullScreen() + 64 (WebFullScreenManager.cpp:208)
10  com.apple.WebKitTestRunner.InjectedBundle	0x0000000772dc390e WTR::InjectedBundlePage::closeFullScreen(OpaqueWKBundlePage const*) + 108 (InjectedBundlePage.cpp:1802)
11  com.apple.WebKitTestRunner.InjectedBundle	0x0000000772dc390e WTR::InjectedBundlePage::closeFullScreen(OpaqueWKBundlePage const*) + 108 (InjectedBundlePage.cpp:1802)
12  com.apple.WebKitTestRunner.InjectedBundle	0x0000000772dc390e WTR::InjectedBundlePage::closeFullScreen(OpaqueWKBundlePage const*) + 108 (InjectedBundlePage.cpp:1802)
13  com.apple.WebKitTestRunner.InjectedBundle	0x0000000772dc390e WTR::InjectedBundlePage::closeFullScreen(OpaqueWKBundlePage const*) + 108 (InjectedBundlePage.cpp:1802)


https://results.webkit.org/?suite=layout-tests&test=fullscreen%2Ffull-screen-remove-ancestor.html
Comment 1 Ryan Haddad 2021-04-01 15:49:44 PDT
Created attachment 424956 [details]
crash log
Comment 2 Ryan Haddad 2021-04-01 15:50:10 PDT
rdar://75009548
Comment 3 Ryan Haddad 2021-04-01 15:50:20 PDT
rdar://75576840
Comment 4 Eric Hutchison 2021-06-28 13:43:14 PDT
*** Bug 227457 has been marked as a duplicate of this bug. ***
Comment 5 Eric Hutchison 2021-07-07 10:25:54 PDT
Created attachment 433042 [details]
Updated Test Expectations
Comment 6 Ryan Haddad 2021-07-07 10:52:33 PDT
Comment on attachment 433042 [details]
Updated Test Expectations

Clearing flags on attachment: 433042

Committed r279649 (239462@main): <https://commits.webkit.org/239462@main>
Comment 7 Jean-Yves Avenard [:jya] 2021-07-29 21:27:52 PDT
Created attachment 434603 [details]
Patch
Comment 8 EWS 2021-07-30 15:38:35 PDT
Committed r280500 (240132@main): <https://commits.webkit.org/240132@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 434603 [details].