[webkit-changes] [WebKit/WebKit] 39ec95: [WebXR] Don't create WebXRSystem when iterating al...
Dan Glastonbury
noreply at github.com
Sun Apr 14 03:22:56 PDT 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 39ec95b06fd161d6480b867020ae4339b8f69e5e
https://github.com/WebKit/WebKit/commit/39ec95b06fd161d6480b867020ae4339b8f69e5e
Author: Dan Glastonbury <djg at apple.com>
Date: 2024-04-13 (Sat, 13 Apr 2024)
Changed paths:
M Source/WebCore/Modules/webxr/NavigatorWebXR.cpp
M Source/WebCore/Modules/webxr/NavigatorWebXR.h
M Source/WebCore/page/Page.cpp
Log Message:
-----------
[WebXR] Don't create WebXRSystem when iterating all DOM objects
https://bugs.webkit.org/show_bug.cgi?id=272564
rdar://124261374
Reviewed by Cameron McCormack.
WebCore::Page::hasActiveImmersiveSession() is creating a new WebXRSystem DOM
object inside WebCore::ScriptExecutionContext::forEachActiveDOMObject(…) which
is forbidden. If there is no WebXRSystem, we should consider that there is no
active immersive session.
* Source/WebCore/Modules/webxr/NavigatorWebXR.cpp:
(WebCore::NavigatorWebXR::xrIfExists):
* Source/WebCore/Modules/webxr/NavigatorWebXR.h:
* Source/WebCore/page/Page.cpp:
(WebCore::Page::hasActiveImmersiveSession const):
Canonical link: https://commits.webkit.org/277468@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list