Skip to content
Permalink
Browse files
Unreviewed, rolling out r202580.
https://bugs.webkit.org/show_bug.cgi?id=159245

Caused all WKTR tests to fail on GuardMalloc and Production
only for unknown reasons, investigating offline. (Requested by
brrian on #webkit).

Reverted changeset:

"RunLoop::Timer should use constructor templates instead of
class templates"
https://bugs.webkit.org/show_bug.cgi?id=159153
http://trac.webkit.org/changeset/202580

Canonical link: https://commits.webkit.org/177358@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@202611 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
webkit-commit-queue committed Jun 29, 2016
1 parent ae3722d commit 4e5ce1505794da0be0848b254a8560803d3bd8e6
Showing with 202 additions and 124 deletions.
  1. +16 −0 Source/JavaScriptCore/ChangeLog
  2. +2 −2 Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.cpp
  3. +16 −0 Source/WTF/ChangeLog
  4. +9 −11 Source/WTF/wtf/RunLoop.h
  5. +16 −0 Source/WebCore/ChangeLog
  6. +2 −2 Source/WebCore/Modules/mediasession/WebMediaSessionManager.cpp
  7. +2 −2 Source/WebCore/Modules/mediasession/WebMediaSessionManager.h
  8. +1 −1 Source/WebCore/page/WheelEventTestTrigger.cpp
  9. +1 −1 Source/WebCore/page/WheelEventTestTrigger.h
  10. +1 −1 Source/WebCore/page/mac/TextIndicatorWindow.h
  11. +1 −1 Source/WebCore/page/mac/TextIndicatorWindow.mm
  12. +1 −1 Source/WebCore/platform/MainThreadSharedTimer.h
  13. +2 −2 Source/WebCore/platform/cocoa/ScrollController.h
  14. +2 −2 Source/WebCore/platform/cocoa/ScrollController.mm
  15. +1 −1 Source/WebCore/platform/glib/MainThreadSharedTimerGLib.cpp
  16. +1 −1 Source/WebCore/platform/graphics/MediaPlaybackTargetPicker.cpp
  17. +1 −1 Source/WebCore/platform/graphics/MediaPlaybackTargetPicker.h
  18. +1 −1 Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp
  19. +1 −1 Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h
  20. +1 −1 Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp
  21. +1 −1 Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.h
  22. +2 −2 Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer.cpp
  23. +2 −2 Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerProxy.cpp
  24. +2 −2 Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerProxy.h
  25. +1 −1 Source/WebCore/platform/mock/MediaPlaybackTargetPickerMock.cpp
  26. +1 −1 Source/WebCore/platform/mock/MediaPlaybackTargetPickerMock.h
  27. +1 −1 Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp
  28. +1 −1 Source/WebCore/platform/mock/MockRealtimeVideoSource.h
  29. +2 −2 Source/WebCore/platform/network/ResourceHandleInternal.h
  30. +16 −0 Source/WebKit2/ChangeLog
  31. +2 −2 Source/WebKit2/NetworkProcess/Downloads/soup/DownloadSoup.cpp
  32. +2 −2 Source/WebKit2/Platform/IPC/mac/ConnectionMac.mm
  33. +2 −2 Source/WebKit2/PluginProcess/PluginControllerProxy.cpp
  34. +2 −2 Source/WebKit2/PluginProcess/PluginControllerProxy.h
  35. +1 −1 Source/WebKit2/PluginProcess/PluginProcess.cpp
  36. +1 −1 Source/WebKit2/PluginProcess/PluginProcess.h
  37. +1 −1 Source/WebKit2/Shared/ChildProcess.cpp
  38. +1 −1 Source/WebKit2/Shared/ChildProcess.h
  39. +1 −1 Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.h
  40. +2 −2 Source/WebKit2/UIProcess/API/Cocoa/APISerializedScriptValueCocoa.mm
  41. +4 −4 Source/WebKit2/UIProcess/API/gtk/WebKitWebViewBase.cpp
  42. +1 −1 Source/WebKit2/UIProcess/DrawingAreaProxy.cpp
  43. +1 −1 Source/WebKit2/UIProcess/DrawingAreaProxy.h
  44. +1 −1 Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp
  45. +1 −1 Source/WebKit2/UIProcess/DrawingAreaProxyImpl.h
  46. +1 −1 Source/WebKit2/UIProcess/Plugins/gtk/PluginInfoCache.cpp
  47. +1 −1 Source/WebKit2/UIProcess/Plugins/gtk/PluginInfoCache.h
  48. +1 −1 Source/WebKit2/UIProcess/ProcessThrottler.cpp
  49. +1 −1 Source/WebKit2/UIProcess/ProcessThrottler.h
  50. +1 −1 Source/WebKit2/UIProcess/ResponsivenessTimer.cpp
  51. +1 −1 Source/WebKit2/UIProcess/ResponsivenessTimer.h
  52. +2 −2 Source/WebKit2/UIProcess/ViewGestureController.cpp
  53. +1 −1 Source/WebKit2/UIProcess/VisitedLinkStore.cpp
  54. +1 −1 Source/WebKit2/UIProcess/VisitedLinkStore.h
  55. +1 −1 Source/WebKit2/UIProcess/WebInspectorProxy.cpp
  56. +1 −1 Source/WebKit2/UIProcess/WebInspectorProxy.h
  57. +1 −1 Source/WebKit2/UIProcess/WebProcessPool.cpp
  58. +1 −1 Source/WebKit2/UIProcess/WebProcessPool.h
  59. +2 −2 Source/WebKit2/UIProcess/gtk/GestureController.cpp
  60. +2 −2 Source/WebKit2/UIProcess/gtk/GestureController.h
  61. +1 −1 Source/WebKit2/UIProcess/gtk/WebPopupMenuProxyGtk.cpp
  62. +1 −1 Source/WebKit2/UIProcess/gtk/WebPopupMenuProxyGtk.h
  63. +2 −2 Source/WebKit2/UIProcess/mac/ViewGestureController.h
  64. +1 −1 Source/WebKit2/WebProcess/Network/WebLoaderStrategy.cpp
  65. +1 −1 Source/WebKit2/WebProcess/Network/WebLoaderStrategy.h
  66. +1 −1 Source/WebKit2/WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp
  67. +1 −1 Source/WebKit2/WebProcess/Plugins/Netscape/NPRuntimeObjectMap.h
  68. +2 −2 Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp
  69. +2 −2 Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.h
  70. +1 −1 Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePluginStream.cpp
  71. +1 −1 Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePluginStream.h
  72. +1 −1 Source/WebKit2/WebProcess/Plugins/PluginView.cpp
  73. +1 −1 Source/WebKit2/WebProcess/Plugins/PluginView.h
  74. +1 −1 Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedDrawingArea.cpp
  75. +1 −1 Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedDrawingArea.h
  76. +1 −1 Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/ThreadedCoordinatedLayerTreeHost.cpp
  77. +1 −1 Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/ThreadedCoordinatedLayerTreeHost.h
  78. +2 −2 Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp
  79. +2 −2 Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.h
  80. +2 −2 Source/WebKit2/WebProcess/WebPage/WebPage.cpp
  81. +2 −2 Source/WebKit2/WebProcess/WebPage/WebPage.h
  82. +1 −1 Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp
  83. +1 −1 Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.h
  84. +1 −1 Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.h
  85. +1 −1 Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm
  86. +16 −0 Tools/ChangeLog
  87. +4 −4 Tools/TestWebKitAPI/Tests/WTF/RunLoop.cpp
  88. +1 −1 Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp
  89. +1 −1 Tools/WebKitTestRunner/InjectedBundle/TestRunner.h
@@ -1,3 +1,19 @@
2016-06-28 Commit Queue <commit-queue@webkit.org>

Unreviewed, rolling out r202580.
https://bugs.webkit.org/show_bug.cgi?id=159245

Caused all WKTR tests to fail on GuardMalloc and Production
only for unknown reasons, investigating offline. (Requested by
brrian on #webkit).

Reverted changeset:

"RunLoop::Timer should use constructor templates instead of
class templates"
https://bugs.webkit.org/show_bug.cgi?id=159153
http://trac.webkit.org/changeset/202580

2016-06-28 Keith Miller <keith_miller@apple.com>

We should not crash there is a finally inside a for-in loop
@@ -55,13 +55,13 @@ class SendGarbageCollectionEventsTask {

HeapFrontendDispatcher& m_frontendDispatcher;
Vector<GarbageCollectionData> m_collections;
RunLoop::Timer m_timer;
RunLoop::Timer<SendGarbageCollectionEventsTask> m_timer;
Lock m_mutex;
};

SendGarbageCollectionEventsTask::SendGarbageCollectionEventsTask(HeapFrontendDispatcher& frontendDispatcher)
: m_frontendDispatcher(frontendDispatcher)
, m_timer(RunLoop::current(), *this, &SendGarbageCollectionEventsTask::timerFired)
, m_timer(RunLoop::current(), this, &SendGarbageCollectionEventsTask::timerFired)
{
}

@@ -1,3 +1,19 @@
2016-06-28 Commit Queue <commit-queue@webkit.org>

Unreviewed, rolling out r202580.
https://bugs.webkit.org/show_bug.cgi?id=159245

Caused all WKTR tests to fail on GuardMalloc and Production
only for unknown reasons, investigating offline. (Requested by
brrian on #webkit).

Reverted changeset:

"RunLoop::Timer should use constructor templates instead of
class templates"
https://bugs.webkit.org/show_bug.cgi?id=159153
http://trac.webkit.org/changeset/202580

2016-06-28 Brian Burg <bburg@apple.com>

RunLoop::Timer should use constructor templates instead of class templates
@@ -1,5 +1,5 @@
/*
* Copyright (C) 2010, 2016 Apple Inc. All rights reserved.
* Copyright (C) 2010 Apple Inc. All rights reserved.
* Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
* Portions Copyright (c) 2010 Motorola Mobility, Inc. All rights reserved.
*
@@ -128,25 +128,23 @@ class RunLoop : public FunctionDispatcher {
#endif
};

template <typename TimerFiredClass>
class Timer : public TimerBase {
public:
template <typename TimerFiredClass, typename TimerFiredBaseClass>
Timer(RunLoop& runLoop, TimerFiredClass& object, void (TimerFiredBaseClass::*function)())
: TimerBase(runLoop)
, m_function(std::bind(function, &object))
{
}
typedef void (TimerFiredClass::*TimerFiredFunction)();

Timer(RunLoop& runLoop, std::function<void ()> function)
Timer(RunLoop& runLoop, TimerFiredClass* o, TimerFiredFunction f)
: TimerBase(runLoop)
, m_function(WTFMove(function))
, m_object(o)
, m_function(f)
{
}

private:
virtual void fired() { m_function(); }
virtual void fired() { (m_object->*m_function)(); }

std::function<void ()> m_function;
TimerFiredClass* m_object;
TimerFiredFunction m_function;
};

class Holder;
@@ -1,3 +1,19 @@
2016-06-28 Commit Queue <commit-queue@webkit.org>

Unreviewed, rolling out r202580.
https://bugs.webkit.org/show_bug.cgi?id=159245

Caused all WKTR tests to fail on GuardMalloc and Production
only for unknown reasons, investigating offline. (Requested by
brrian on #webkit).

Reverted changeset:

"RunLoop::Timer should use constructor templates instead of
class templates"
https://bugs.webkit.org/show_bug.cgi?id=159153
http://trac.webkit.org/changeset/202580

2016-06-28 Benjamin Poulain <benjamin@webkit.org>

Rename ChildrenAffectedByActive to StyleAffectedByActive
@@ -127,8 +127,8 @@ WebCore::MediaPlaybackTargetPicker& WebMediaSessionManager::targetPicker()
}

WebMediaSessionManager::WebMediaSessionManager()
: m_taskTimer(RunLoop::current(), *this, &WebMediaSessionManager::taskTimerFired)
, m_watchdogTimer(RunLoop::current(), *this, &WebMediaSessionManager::watchdogTimerFired)
: m_taskTimer(RunLoop::current(), this, &WebMediaSessionManager::taskTimerFired)
, m_watchdogTimer(RunLoop::current(), this, &WebMediaSessionManager::watchdogTimerFired)
{
}

@@ -96,8 +96,8 @@ class WebMediaSessionManager : public MediaPlaybackTargetPicker::Client {

void watchdogTimerFired();

RunLoop::Timer m_taskTimer;
RunLoop::Timer m_watchdogTimer;
RunLoop::Timer<WebMediaSessionManager> m_taskTimer;
RunLoop::Timer<WebMediaSessionManager> m_watchdogTimer;

Vector<std::unique_ptr<ClientState>> m_clientState;
RefPtr<MediaPlaybackTarget> m_playbackTarget;
@@ -39,7 +39,7 @@
namespace WebCore {

WheelEventTestTrigger::WheelEventTestTrigger()
: m_testTriggerTimer(RunLoop::current(), *this, &WheelEventTestTrigger::triggerTestTimerFired)
: m_testTriggerTimer(RunLoop::current(), this, &WheelEventTestTrigger::triggerTestTimerFired)
{
}

@@ -59,7 +59,7 @@ class WheelEventTestTrigger : public ThreadSafeRefCounted<WheelEventTestTrigger>

private:
std::function<void()> m_testNotificationCallback;
RunLoop::Timer m_testTriggerTimer;
RunLoop::Timer<WheelEventTestTrigger> m_testTriggerTimer;
mutable Lock m_testTriggerMutex;
WTF::HashMap<ScrollableAreaIdentifier, std::set<DeferTestTriggerReason>> m_deferTestTriggerReasons;
};
@@ -74,7 +74,7 @@ class TextIndicatorWindow {
RetainPtr<NSWindow> m_textIndicatorWindow;
RetainPtr<WebTextIndicatorView> m_textIndicatorView;

RunLoop::Timer m_temporaryTextIndicatorTimer;
RunLoop::Timer<TextIndicatorWindow> m_temporaryTextIndicatorTimer;
};

#endif // PLATFORM(MAC)
@@ -404,7 +404,7 @@ - (BOOL)isFlipped

TextIndicatorWindow::TextIndicatorWindow(NSView *targetView)
: m_targetView(targetView)
, m_temporaryTextIndicatorTimer(RunLoop::main(), *this, &TextIndicatorWindow::startFadeOut)
, m_temporaryTextIndicatorTimer(RunLoop::main(), this, &TextIndicatorWindow::startFadeOut)
{
}

@@ -55,7 +55,7 @@ class MainThreadSharedTimer final : public SharedTimer {

std::function<void()> m_firedFunction;
#if PLATFORM(GTK)
RunLoop::Timer m_timer;
RunLoop::Timer<MainThreadSharedTimer> m_timer;
#endif
};

@@ -177,7 +177,7 @@ class ScrollController {
CFTimeInterval m_startTime { 0 };
FloatSize m_startStretch;
FloatSize m_origVelocity;
RunLoop::Timer m_snapRubberbandTimer;
RunLoop::Timer<ScrollController> m_snapRubberbandTimer;
#endif

#if ENABLE(CSS_SCROLL_SNAP)
@@ -187,7 +187,7 @@ class ScrollController {
std::unique_ptr<ScrollSnapAnimatorState> m_verticalScrollSnapState;
std::unique_ptr<ScrollSnapAnimationCurveState> m_scrollSnapCurveState;
#if PLATFORM(MAC)
RunLoop::Timer m_scrollSnapTimer;
RunLoop::Timer<ScrollController> m_scrollSnapTimer;
#endif
#endif

@@ -121,10 +121,10 @@ static float scrollWheelMultiplier()
ScrollController::ScrollController(ScrollControllerClient& client)
: m_client(client)
#if ENABLE(RUBBER_BANDING)
, m_snapRubberbandTimer(RunLoop::current(), *this, &ScrollController::snapRubberBandTimerFired)
, m_snapRubberbandTimer(RunLoop::current(), this, &ScrollController::snapRubberBandTimerFired)
#endif
#if ENABLE(CSS_SCROLL_SNAP) && PLATFORM(MAC)
, m_scrollSnapTimer(RunLoop::current(), *this, &ScrollController::scrollSnapTimerFired)
, m_scrollSnapTimer(RunLoop::current(), this, &ScrollController::scrollSnapTimerFired)
#endif
{
}
@@ -33,7 +33,7 @@
namespace WebCore {

MainThreadSharedTimer::MainThreadSharedTimer()
: m_timer(RunLoop::main(), *this, &MainThreadSharedTimer::fired)
: m_timer(RunLoop::main(), this, &MainThreadSharedTimer::fired)
{
// This is GDK_PRIORITY_REDRAW, but we don't want to depend on GDK here just to use a constant.
m_timer.setPriority(G_PRIORITY_HIGH_IDLE + 20);
@@ -37,7 +37,7 @@ static const double pendingActionInterval = 1.0 / 10.0;

MediaPlaybackTargetPicker::MediaPlaybackTargetPicker(Client& client)
: m_client(&client)
, m_pendingActionTimer(RunLoop::main(), *this, &MediaPlaybackTargetPicker::pendingActionTimerFired)
, m_pendingActionTimer(RunLoop::main(), this, &MediaPlaybackTargetPicker::pendingActionTimerFired)
{
}

@@ -79,7 +79,7 @@ class MediaPlaybackTargetPicker {

PendingActionFlags m_pendingActionFlags { 0 };
Client* m_client;
RunLoop::Timer m_pendingActionTimer;
RunLoop::Timer<MediaPlaybackTargetPicker> m_pendingActionTimer;
};

} // namespace WebCore
@@ -163,7 +163,7 @@ MediaPlayerPrivateGStreamer::MediaPlayerPrivateGStreamer(MediaPlayer* player)
, m_volumeAndMuteInitialized(false)
, m_hasVideo(false)
, m_hasAudio(false)
, m_readyTimerHandler(RunLoop::main(), *this, &MediaPlayerPrivateGStreamer::readyTimerFired)
, m_readyTimerHandler(RunLoop::main(), this, &MediaPlayerPrivateGStreamer::readyTimerFired)
, m_totalBytes(0)
, m_preservesPitch(false)
#if ENABLE(WEB_AUDIO)
@@ -231,7 +231,7 @@ class MediaPlayerPrivateGStreamer : public MediaPlayerPrivateGStreamerBase {
bool m_volumeAndMuteInitialized;
bool m_hasVideo;
bool m_hasAudio;
RunLoop::Timer m_readyTimerHandler;
RunLoop::Timer<MediaPlayerPrivateGStreamer> m_readyTimerHandler;
mutable unsigned long long m_totalBytes;
URL m_url;
bool m_preservesPitch;
@@ -150,7 +150,7 @@ MediaPlayerPrivateGStreamerBase::MediaPlayerPrivateGStreamerBase(MediaPlayer* pl
, m_readyState(MediaPlayer::HaveNothing)
, m_networkState(MediaPlayer::Empty)
#if USE(GSTREAMER_GL)
, m_drawTimer(RunLoop::main(), *this, &MediaPlayerPrivateGStreamerBase::repaint)
, m_drawTimer(RunLoop::main(), this, &MediaPlayerPrivateGStreamerBase::repaint)
#endif
, m_usingFallbackVideoSink(false)
, m_videoSourceRotation(NoVideoSourceRotation)
@@ -182,7 +182,7 @@ class MediaPlayerPrivateGStreamerBase : public MediaPlayerPrivateInterface
mutable GMutex m_sampleMutex;
GRefPtr<GstSample> m_sample;
#if USE(GSTREAMER_GL)
RunLoop::Timer m_drawTimer;
RunLoop::Timer<MediaPlayerPrivateGStreamerBase> m_drawTimer;
#endif
mutable FloatSize m_videoSize;
bool m_usingFallbackVideoSink;
@@ -74,7 +74,7 @@ class VideoRenderRequestScheduler {
public:
VideoRenderRequestScheduler()
#if !USE(COORDINATED_GRAPHICS_THREADED)
: m_timer(RunLoop::main(), *this, &VideoRenderRequestScheduler::render)
: m_timer(RunLoop::main(), this, &VideoRenderRequestScheduler::render)
#endif
{
#if PLATFORM(GTK) && !USE(COORDINATED_GRAPHICS_THREADED)
@@ -140,7 +140,7 @@ class VideoRenderRequestScheduler {
GRefPtr<GstSample> m_sample;
#if !USE(COORDINATED_GRAPHICS_THREADED)
RunLoop::Timer m_timer;
RunLoop::Timer<VideoRenderRequestScheduler> m_timer;
Condition m_dataCondition;
GRefPtr<WebKitVideoSink> m_sink;
#endif
@@ -41,7 +41,7 @@ namespace WebCore {
TextureMapperPlatformLayerProxy::TextureMapperPlatformLayerProxy()
: m_compositor(nullptr)
, m_targetLayer(nullptr)
, m_releaseUnusedBuffersTimer(RunLoop::current(), *this, &TextureMapperPlatformLayerProxy::releaseUnusedBuffersTimerFired)
, m_releaseUnusedBuffersTimer(RunLoop::current(), this, &TextureMapperPlatformLayerProxy::releaseUnusedBuffersTimerFired)
{
}

@@ -66,7 +66,7 @@ void TextureMapperPlatformLayerProxy::activateOnCompositingThread(Compositor* co
if (m_targetLayer && m_currentBuffer)
m_targetLayer->setContentsLayer(m_currentBuffer.get());

m_compositorThreadUpdateTimer = std::make_unique<RunLoop::Timer>(RunLoop::current(), *this, &TextureMapperPlatformLayerProxy::compositorThreadUpdateTimerFired);
m_compositorThreadUpdateTimer = std::make_unique<RunLoop::Timer<TextureMapperPlatformLayerProxy>>(RunLoop::current(), this, &TextureMapperPlatformLayerProxy::compositorThreadUpdateTimerFired);
}

void TextureMapperPlatformLayerProxy::invalidate()
@@ -93,13 +93,13 @@ class TextureMapperPlatformLayerProxy : public ThreadSafeRefCounted<TextureMappe

Vector<std::unique_ptr<TextureMapperPlatformLayerBuffer>> m_usedBuffers;

RunLoop::Timer m_releaseUnusedBuffersTimer;
RunLoop::Timer<TextureMapperPlatformLayerProxy> m_releaseUnusedBuffersTimer;
#ifndef NDEBUG
ThreadIdentifier m_compositorThreadID { 0 };
#endif

void compositorThreadUpdateTimerFired();
std::unique_ptr<RunLoop::Timer> m_compositorThreadUpdateTimer;
std::unique_ptr<RunLoop::Timer<TextureMapperPlatformLayerProxy>> m_compositorThreadUpdateTimer;
std::function<void()> m_compositorThreadUpdateFunction;
};

@@ -41,7 +41,7 @@ static const double timerInterval = 1.0 / 10.0;

MediaPlaybackTargetPickerMock::MediaPlaybackTargetPickerMock(MediaPlaybackTargetPicker::Client& client)
: MediaPlaybackTargetPicker(client)
, m_timer(RunLoop::main(), *this, &MediaPlaybackTargetPickerMock::timerFired)
, m_timer(RunLoop::main(), this, &MediaPlaybackTargetPickerMock::timerFired)
{
LOG(Media, "MediaPlaybackTargetPickerMock::MediaPlaybackTargetPickerMock");
}
@@ -55,7 +55,7 @@ class MediaPlaybackTargetPickerMock final : public MediaPlaybackTargetPicker {
void timerFired();

String m_deviceName;
RunLoop::Timer m_timer;
RunLoop::Timer<MediaPlaybackTargetPickerMock> m_timer;
MediaPlaybackTargetContext::State m_state { MediaPlaybackTargetContext::Unknown };
bool m_showingMenu { false };
};
@@ -64,7 +64,7 @@ Ref<MockRealtimeVideoSource> MockRealtimeVideoSource::createMuted(const String&

MockRealtimeVideoSource::MockRealtimeVideoSource(const String& name)
: MockRealtimeMediaSource(createCanonicalUUIDString(), RealtimeMediaSource::Video, name)
, m_timer(RunLoop::current(), *this, &MockRealtimeVideoSource::generateFrame)
, m_timer(RunLoop::current(), this, &MockRealtimeVideoSource::generateFrame)
{
m_dashWidths.reserveInitialCapacity(2);
m_dashWidths.uncheckedAppend(6);
@@ -103,7 +103,7 @@ class MockRealtimeVideoSource : public MockRealtimeMediaSource {
unsigned m_frameRate { 30 };
unsigned m_frameNumber { 0 };

RunLoop::Timer m_timer;
RunLoop::Timer<MockRealtimeVideoSource> m_timer;
};

} // namespace WebCore
@@ -91,7 +91,7 @@ namespace WebCore {
#endif
#if USE(SOUP)
, m_cancelled(false)
, m_timeoutSource(RunLoop::main(), *loader, &ResourceHandle::timeoutFired)
, m_timeoutSource(RunLoop::main(), loader, &ResourceHandle::timeoutFired)
, m_bodySize(0)
, m_bodyDataSent(0)
, m_redirectCount(0)
@@ -170,7 +170,7 @@ namespace WebCore {
GRefPtr<SoupMultipartInputStream> m_multipartInputStream;
GRefPtr<GCancellable> m_cancellable;
GRefPtr<GAsyncResult> m_deferredResult;
RunLoop::Timer m_timeoutSource;
RunLoop::Timer<ResourceHandle> m_timeoutSource;
GUniquePtr<SoupBuffer> m_soupBuffer;
unsigned long m_bodySize;
unsigned long m_bodyDataSent;

0 comments on commit 4e5ce15

Please sign in to comment.