Skip to content

Commit

Permalink
Non-unified build fixes, late February 2024 edition
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=270156

Unreviewed non-unified build fixes.

* Source/JavaScriptCore/assembler/PerfLog.cpp: Add missing inclusions of
Options.h and wtf/StringPrintStream.h.
* Source/WebCore/Modules/mediasource/MediaSourceInterfaceMainThread.cpp:
Add missing inclusions of SourceBufferList.h and TimeRanges.h.
* Source/WebCore/Modules/mediasource/MediaSourceInterfaceMainThread.h:
Add missing forward declaration for the WebCore::MediaSource type.
* Source/WebCore/Modules/mediasource/SourceBuffer.cpp: Remove unused
ExponentialMovingAverageCoefficient constant.
* Source/WebCore/Modules/mediasource/SourceBufferList.cpp: Add missing
ContextDestructionObserverInlines.h header.
* Source/WebCore/Modules/mediastream/MediaStreamTrackProcessor.cpp:
Ditto, plus wtf/IsoMallocInlines.h.
* Source/WebCore/Modules/mediastream/RTCDTMFSender.cpp: Add missing
ContextDestructionObserverInlines.h.
* Source/WebCore/Modules/mediastream/RTCRtpSFrameTransform.cpp: Ditto
* Source/WebCore/Modules/mediastream/RTCRtpSender.cpp: Ditto.
* Source/WebCore/Modules/speech/SpeechSynthesisUtterance.cpp: Ditto,
plus move all inclusions into ENABLE(SPEECH_SYNTHESIS).
* Source/WebCore/Modules/webcodecs/WebCodecsAudioDecoder.cpp: Add
missing ContextDestructionObserverInlines.h inclusion.
* Source/WebCore/Modules/webcodecs/WebCodecsAudioEncoder.cpp: Ditto.
* Source/WebCore/Modules/webcodecs/WebCodecsVideoDecoder.cpp: Ditto.
* Source/WebCore/Modules/webcodecs/WebCodecsVideoEncoder.cpp: Ditto.
* Source/WebCore/animation/StyleOriginatedAnimationEvent.cpp: Add
missing Node.h inclusion.
* Source/WebCore/animation/WebAnimationUtilities.h: Add missing forward
declarations for WebCore::Document and WebCore::Style::PseudoElementIdentifier.
* Source/WebCore/bindings/js/GCController.h: Add missing forward
declaration for the JSC::VM type.
* Source/WebCore/css/parser/CSSSelectorParser.cpp: Add missing
CSSTokenizer.h inclusion.
* Source/WebCore/dom/FragmentDirectiveGenerator.cpp: Add missing
Document.h inclusion.
* Source/WebCore/dom/FragmentDirectiveGenerator.h: Add missing wtf/URL.h
inclusion, add missing WebCore::SimpleRange forward declaration, and
remove the unneeded WebCore::Range one.
* Source/WebCore/editing/VisibleSelection.cpp: Replace Element.h
inclusion with ElementInlines.h.
* Source/WebCore/html/MediaElementSession.cpp: Add missing Navigator.h
inclusion.
* Source/WebCore/loader/cache/CachedResourceRequest.cpp: Add missing
Quirks.h inclusion.
* Source/WebCore/page/PointerLockController.cpp: Add missing
DocumentInlines.h inclusion.
* Source/WebCore/page/Quirks.h: Add missing WebCore::LocalFrame forward
declaration.
* Source/WebCore/platform/MediaDescription.h: Add missing
wtf/text/WTFString.h inclusion, remove the unneeded wtf/Forward.h one.
* Source/WebCore/rendering/RenderViewTransitionCapture.cpp: Add missing
inclusions for ImageOverlayController.h, ImageQualityController.h,
PaintInfo.h, and wtf/IsoMallocInlines.h.
* Source/WebCore/rendering/svg/RenderSVGInlineText.cpp: Add missing
inclusion of RenderObjectInlines.h.
* Source/WebCore/style/Styleable.cpp: Add missing RenderView.h
inclusion.
* Source/WebKit/NetworkProcess/Cookies/WebCookieManager.cpp: Add missing
Logging.h inclusion.
* Source/WebKit/NetworkProcess/cache/NetworkCacheDataGLib.cpp: Add
missing WebCore:: namespace prefixes to uses of the
WebCore::SharedMemory type.
(WebKit::NetworkCache::Data::tryCreateSharedMemory const):
* Source/WebKit/Shared/WebCompiledContentRuleListData.cpp: Ditto.
(WebKit::WebCompiledContentRuleListData::createDataHandle const):
(WebKit::WebCompiledContentRuleListData::WebCompiledContentRuleListData):
* Source/WebKit/Shared/gtk/WebEventFactory.cpp: Remove unused.
gtkScrollDeltaMultiplier constant.
* Source/WebKit/UIProcess/WebProcessProxy.cpp: Add missing inclusions
for APIPageConfiguration.h and wtf/Scope.h.
* Source/WebKit/UIProcess/gtk/ViewGestureControllerGtk.cpp: Remove
unused swipeOverlayShadowOpacity constant.

Canonical link: https://commits.webkit.org/275381@main
  • Loading branch information
aperezdc committed Feb 27, 2024
1 parent bbf768d commit 3f4bd57
Show file tree
Hide file tree
Showing 35 changed files with 52 additions and 19 deletions.
2 changes: 2 additions & 0 deletions Source/JavaScriptCore/assembler/PerfLog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@

#if ENABLE(ASSEMBLER) && (OS(LINUX) || OS(DARWIN))

#include "Options.h"
#include <array>
#include <fcntl.h>
#include <mutex>
Expand All @@ -41,6 +42,7 @@
#include <wtf/MonotonicTime.h>
#include <wtf/PageBlock.h>
#include <wtf/ProcessID.h>
#include <wtf/StringPrintStream.h>
#include <wtf/TZoneMallocInlines.h>

namespace JSC {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@

#include "ManagedMediaSource.h"
#include "MediaSource.h"
#include "SourceBufferList.h"
#include "TimeRanges.h"

namespace WebCore {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@

namespace WebCore {

class MediaSource;

class MediaSourceInterfaceMainThread : public MediaSourceInterfaceProxy {
public:
static Ref<MediaSourceInterfaceMainThread> create(Ref<MediaSource>&& mediaSource) { return adoptRef(*new MediaSourceInterfaceMainThread(WTFMove(mediaSource))); }
Expand Down
3 changes: 0 additions & 3 deletions Source/WebCore/Modules/mediasource/SourceBuffer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,6 @@ namespace WebCore {

WTF_MAKE_ISO_ALLOCATED_IMPL(SourceBuffer);

static const double ExponentialMovingAverageCoefficient = 0.2;


class SourceBufferClientImpl final : public SourceBufferPrivateClient {
public:
static Ref<SourceBufferClientImpl> create(SourceBuffer& parent) { return adoptRef(*new SourceBufferClientImpl(parent)); }
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/mediasource/SourceBufferList.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@

#if ENABLE(MEDIA_SOURCE)

#include "ContextDestructionObserverInlines.h"
#include "Event.h"
#include "EventNames.h"
#include "SourceBuffer.h"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,11 @@

#if ENABLE(MEDIA_STREAM) && ENABLE(WEB_CODECS)

#include "ContextDestructionObserverInlines.h"
#include "JSWebCodecsVideoFrame.h"
#include "Logging.h"
#include "ReadableStream.h"
#include <wtf/IsoMallocInlines.h>
#include <wtf/Seconds.h>

namespace WebCore {
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/mediastream/RTCDTMFSender.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

#if ENABLE(WEB_RTC)

#include "ContextDestructionObserverInlines.h"
#include "RTCDTMFSenderBackend.h"
#include "RTCDTMFToneChangeEvent.h"
#include "RTCRtpSender.h"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

#if ENABLE(WEB_RTC)

#include "ContextDestructionObserverInlines.h"
#include "CryptoKeyRaw.h"
#include "JSDOMConvertBufferSource.h"
#include "JSDOMPromiseDeferred.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/mediastream/RTCRtpSender.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@

#if ENABLE(WEB_RTC)

#include "ContextDestructionObserverInlines.h"
#include "JSDOMPromiseDeferred.h"
#include "Logging.h"
#include "RTCDTMFSender.h"
Expand Down
6 changes: 3 additions & 3 deletions Source/WebCore/Modules/speech/SpeechSynthesisUtterance.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,12 @@
#include "config.h"
#include "SpeechSynthesisUtterance.h"

#if ENABLE(SPEECH_SYNTHESIS)

#include "ContextDestructionObserverInlines.h"
#include "EventNames.h"
#include "SpeechSynthesisErrorEvent.h"
#include "SpeechSynthesisEvent.h"

#if ENABLE(SPEECH_SYNTHESIS)

#include <wtf/IsoMallocInlines.h>

namespace WebCore {
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/webcodecs/WebCodecsAudioDecoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@

#if ENABLE(WEB_CODECS)

#include "ContextDestructionObserverInlines.h"
#include "DOMException.h"
#include "Event.h"
#include "EventNames.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/webcodecs/WebCodecsAudioEncoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
#if ENABLE(WEB_CODECS)

#include "AacEncoderConfig.h"
#include "ContextDestructionObserverInlines.h"
#include "DOMException.h"
#include "Event.h"
#include "EventNames.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/webcodecs/WebCodecsVideoDecoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

#if ENABLE(WEB_CODECS)

#include "ContextDestructionObserverInlines.h"
#include "DOMException.h"
#include "Event.h"
#include "EventNames.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/Modules/webcodecs/WebCodecsVideoEncoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

#if ENABLE(WEB_CODECS)

#include "ContextDestructionObserverInlines.h"
#include "DOMException.h"
#include "Event.h"
#include "EventNames.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/animation/StyleOriginatedAnimationEvent.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
#include "config.h"
#include "StyleOriginatedAnimationEvent.h"

#include "Node.h"
#include "WebAnimationUtilities.h"

#include <wtf/IsoMallocInlines.h>
Expand Down
5 changes: 5 additions & 0 deletions Source/WebCore/animation/WebAnimationUtilities.h
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,14 @@ namespace WebCore {
enum class PseudoId : uint32_t;

class AnimationEventBase;
class Document;
class Element;
class WebAnimation;

namespace Style {
struct PseudoElementIdentifier;
}

inline double secondsToWebAnimationsAPITime(const Seconds time)
{
// Precision of time values
Expand Down
4 changes: 4 additions & 0 deletions Source/WebCore/bindings/js/GCController.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@
#include <wtf/Forward.h>
#include <wtf/Noncopyable.h>

namespace JSC {
class VM;
}

namespace WebCore {

class GCController {
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/css/parser/CSSSelectorParser.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
#include "CSSParserIdioms.h"
#include "CSSSelector.h"
#include "CSSSelectorInlines.h"
#include "CSSTokenizer.h"
#include "CommonAtomStrings.h"
#include "Document.h"
#include "MutableCSSSelector.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/dom/FragmentDirectiveGenerator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
#include "config.h"
#include "FragmentDirectiveGenerator.h"

#include "Document.h"
#include "HTMLParserIdioms.h"
#include "Logging.h"
#include "Range.h"
Expand Down
3 changes: 2 additions & 1 deletion Source/WebCore/dom/FragmentDirectiveGenerator.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,12 @@
#pragma once

#include <wtf/Forward.h>
#include <wtf/URL.h>
#include <wtf/text/WTFString.h>

namespace WebCore {

class Range;
struct SimpleRange;

struct TextDirective {
String textStart;
Expand Down
2 changes: 1 addition & 1 deletion Source/WebCore/editing/VisibleSelection.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
#include "Document.h"
#include "DocumentInlines.h"
#include "Editing.h"
#include "Element.h"
#include "ElementInlines.h"
#include "HTMLInputElement.h"
#include "Settings.h"
#include "ShadowRoot.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/html/MediaElementSession.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
#include "LocalFrameView.h"
#include "Logging.h"
#include "MediaUsageInfo.h"
#include "Navigator.h"
#include "NowPlayingInfo.h"
#include "Page.h"
#include "PlatformMediaSessionManager.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/loader/cache/CachedResourceRequest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
#include "MIMETypeRegistry.h"
#include "MemoryCache.h"
#include "OriginAccessPatterns.h"
#include "Quirks.h"
#include "SecurityPolicy.h"
#include "ServiceWorkerRegistrationData.h"
#include <wtf/NeverDestroyed.h>
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/page/PointerLockController.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@

#include "Chrome.h"
#include "ChromeClient.h"
#include "DocumentInlines.h"
#include "Element.h"
#include "Event.h"
#include "EventNames.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/page/Quirks.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ class EventTypeInfo;
class HTMLElement;
class HTMLVideoElement;
class LayoutUnit;
class LocalFrame;
class PlatformMouseEvent;
class RegistrableDomain;
class SecurityOriginData;
Expand Down
2 changes: 1 addition & 1 deletion Source/WebCore/platform/MediaDescription.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@
#ifndef MediaDescription_h
#define MediaDescription_h

#include <wtf/Forward.h>
#include <wtf/ThreadSafeRefCounted.h>
#include <wtf/text/WTFString.h>

namespace WebCore {

Expand Down
4 changes: 4 additions & 0 deletions Source/WebCore/rendering/RenderViewTransitionCapture.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,11 @@
#include "config.h"
#include "RenderViewTransitionCapture.h"

#include "ImageOverlayController.h"
#include "ImageQualityController.h"
#include "PaintInfo.h"
#include "RenderBoxModelObjectInlines.h"
#include <wtf/IsoMallocInlines.h>

namespace WebCore {

Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/rendering/svg/RenderSVGInlineText.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
#include "LegacyRenderSVGRoot.h"
#include "RenderAncestorIterator.h"
#include "RenderBlock.h"
#include "RenderObjectInlines.h"
#include "RenderSVGText.h"
#include "SVGElementTypeHelpers.h"
#include "SVGInlineTextBoxInlines.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebCore/style/Styleable.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@
#include "RenderListItem.h"
#include "RenderListMarker.h"
#include "RenderStyleInlines.h"
#include "RenderView.h"
#include "StyleCustomPropertyData.h"
#include "StyleOriginatedAnimation.h"
#include "StylePropertyShorthand.h"
Expand Down
1 change: 1 addition & 0 deletions Source/WebKit/NetworkProcess/Cookies/WebCookieManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
#include "config.h"
#include "WebCookieManager.h"

#include "Logging.h"
#include "MessageSenderInlines.h"
#include "NetworkProcess.h"
#include "NetworkProcessProxyMessages.h"
Expand Down
4 changes: 2 additions & 2 deletions Source/WebKit/NetworkProcess/cache/NetworkCacheDataGLib.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -137,15 +137,15 @@ Data Data::adoptMap(FileSystem::MappedFileData&& mappedFile, FileSystem::Platfor
return { adoptGRef(g_bytes_new_with_free_func(map, size, reinterpret_cast<GDestroyNotify>(deleteMapWrapper), wrapper)), fd };
}

RefPtr<SharedMemory> Data::tryCreateSharedMemory() const
RefPtr<WebCore::SharedMemory> Data::tryCreateSharedMemory() const
{
if (isNull() || !isMap())
return nullptr;

int fd = FileSystem::posixFileDescriptor(m_fileDescriptor);
gsize length;
const auto* data = g_bytes_get_data(m_buffer.get(), &length);
return SharedMemory::wrapMap(const_cast<void*>(data), length, fd);
return WebCore::SharedMemory::wrapMap(const_cast<void*>(data), length, fd);
}

} // namespace NetworkCache
Expand Down
4 changes: 2 additions & 2 deletions Source/WebKit/Shared/WebCompiledContentRuleListData.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,14 +37,14 @@ static size_t ruleListDataSize(size_t topURLFiltersBytecodeOffset, size_t topURL
return topURLFiltersBytecodeOffset + topURLFiltersBytecodeSize;
}

std::optional<SharedMemoryHandle> WebCompiledContentRuleListData::createDataHandle(SharedMemory::Protection protection) const
std::optional<WebCore::SharedMemoryHandle> WebCompiledContentRuleListData::createDataHandle(WebCore::SharedMemory::Protection protection) const
{
return data->createHandle(protection);
}

WebCompiledContentRuleListData::WebCompiledContentRuleListData(String&& identifier, std::optional<WebCore::SharedMemoryHandle>&& dataHandle, size_t actionsOffset, size_t actionsSize, size_t urlFiltersBytecodeOffset, size_t urlFiltersBytecodeSize, size_t topURLFiltersBytecodeOffset, size_t topURLFiltersBytecodeSize, size_t frameURLFiltersBytecodeOffset, size_t frameURLFiltersBytecodeSize)
: identifier(WTFMove(identifier))
, data(dataHandle ? SharedMemory::map(WTFMove(*dataHandle), SharedMemory::Protection::ReadOnly) : nullptr)
, data(dataHandle ? WebCore::SharedMemory::map(WTFMove(*dataHandle), WebCore::SharedMemory::Protection::ReadOnly) : nullptr)
, actionsOffset(actionsOffset)
, actionsSize(actionsSize)
, urlFiltersBytecodeOffset(urlFiltersBytecodeOffset)
Expand Down
5 changes: 0 additions & 5 deletions Source/WebKit/Shared/gtk/WebEventFactory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,6 @@ namespace WebKit {

using namespace WebCore;

#if GTK_CHECK_VERSION(4, 7, 0)
// Keep in sync with https://gitlab.gnome.org/GNOME/gtk/-/blob/493660a296af3b8a140714988ddece4199818a04/gtk/gtkscrolledwindow.c#L204
static const double gtkScrollDeltaMultiplier = 2.5;
#endif

static inline bool isGdkKeyCodeFromKeyPad(unsigned keyval)
{
return keyval >= GDK_KEY_KP_Space && keyval <= GDK_KEY_KP_9;
Expand Down
2 changes: 2 additions & 0 deletions Source/WebKit/UIProcess/WebProcessProxy.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
#include "WebProcessProxy.h"

#include "APIFrameHandle.h"
#include "APIPageConfiguration.h"
#include "APIPageHandle.h"
#include "APIUIClient.h"
#include "AuthenticatorManager.h"
Expand Down Expand Up @@ -94,6 +95,7 @@
#include <wtf/NeverDestroyed.h>
#include <wtf/ProcessPrivilege.h>
#include <wtf/RunLoop.h>
#include <wtf/Scope.h>
#include <wtf/URL.h>
#include <wtf/Vector.h>
#include <wtf/WeakListHashSet.h>
Expand Down
1 change: 0 additions & 1 deletion Source/WebKit/UIProcess/gtk/ViewGestureControllerGtk.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ static const double swipeCancelArea = 0.5;
static const double swipeCancelVelocityThreshold = 0.4;

#if USE(GTK4)
static const float swipeOverlayShadowOpacity = 0.06;
static const float swipeOverlayBorderOpacity = 0.05;
static const float swipeOverlayOutlineOpacity = 0.05;
static const float swipeOverlayDimmingOpacity = 0.12;
Expand Down

0 comments on commit 3f4bd57

Please sign in to comment.