NEW 262515
Fix build break for ENABLE(MEDIA_SOURCE) && !ENABLE(MANAGED_MEDIA_SOURCE) after https://commits.webkit.org/268711@main
https://bugs.webkit.org/show_bug.cgi?id=262515
Summary Fix build break for ENABLE(MEDIA_SOURCE) && !ENABLE(MANAGED_MEDIA_SOURCE) aft...
Jigen Zhou
Reported 2023-10-02 16:56:41 PDT
The code change in https://commits.webkit.org/268711@main works if both ENABLE(MEDIA_SOURCE) && ENABLE(MANAGED_MEDIA_SOURCE), but a compiling error can be caused by the generated code.
Attachments
Tomoki Imai
Comment 1 2023-10-05 01:25:10 PDT
The following code is a part of WebCore/DerivedSources/JSVideoTrack.cpp. It accesses "managedMediaSourceEnabled" without "#if ENABLE(ManagedMediaSourceEnabled)" block and causes a build failure if ENABLE(MEDIA_SOURCE) && !ENABLE(ManagedMediaSourceEnabled). #if ENABLE(MEDIA_SOURCE) if (!(downcast<Document>(jsCast<JSDOMGlobalObject*>(globalObject())->scriptExecutionContext())->settingsValues().mediaSourceEnabled || downcast<Document>(jsCast<JSDOMGlobalObject*>(globalObject())->scriptExecutionContext())->settingsValues().managedMediaSourceEnabled)) { hasDisabledRuntimeProperties = true; auto propertyName = Identifier::fromString(vm, "sourceBuffer"_s); VM::DeletePropertyModeScope scope(vm, VM::DeletePropertyMode::IgnoreConfigurable); DeletePropertySlot slot; JSObject::deleteProperty(this, globalObject(), propertyName, slot); } #endif
Tomoki Imai
Comment 2 2023-10-05 01:35:45 PDT
In WebCore/DerivedSources/Settings.h, "managedMediaSourceEnabled" is blocked by " ENABLE(MANAGED_MEDIA_SOURCE)". class Settings : public SettingsBase, public RefCounted<Settings> { struct Values { ... #if ENABLE(MANAGED_MEDIA_SOURCE) bool managedMediaSourceEnabled : 1; #endif
Radar WebKit Bug Importer
Comment 3 2023-10-09 16:57:19 PDT
Note You need to log in before you can comment on or make changes to this bug.