Skip to content

Commit 1bde52c

Browse files
committed
Fix build with RELEASE_LOG disabled.
Remove when https://bugs.webkit.org/show_bug.cgi?id=275607 is fixed upstream.
1 parent ccde30a commit 1bde52c

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,10 +187,12 @@ MediaPlayerPrivateGStreamer::MediaPlayerPrivateGStreamer(MediaPlayer* player)
187187

188188
// MediaPlayer relies on the Document logger, so to prevent duplicate messages in case
189189
// more than one MediaPlayer is created, we register a single observer.
190+
#if !RELEASE_LOG_DISABLED
190191
if (auto player = m_player.get()) {
191192
auto& logObserver = mediaLogObserverSingleton();
192193
logObserver.addWatch(player->mediaPlayerLogger());
193194
}
195+
#endif
194196

195197
#if USE(GLIB)
196198
m_pausedTimerHandler.setPriority(G_PRIORITY_DEFAULT_IDLE);
@@ -333,10 +335,12 @@ void MediaPlayerPrivateGStreamer::registerMediaEngine(MediaEngineRegistrar regis
333335
void MediaPlayerPrivateGStreamer::mediaPlayerWillBeDestroyed()
334336
{
335337
GST_DEBUG_OBJECT(m_pipeline.get(), "Parent MediaPlayer is about to be destroyed");
338+
#if !RELEASE_LOG_DISABLED
336339
if (auto player = m_player.get()) {
337340
auto& logObserver = mediaLogObserverSingleton();
338341
logObserver.removeWatch(player->mediaPlayerLogger());
339342
}
343+
#endif
340344
}
341345

342346
void MediaPlayerPrivateGStreamer::load(const String& urlString)
@@ -3055,8 +3059,11 @@ void MediaPlayerPrivateGStreamer::createGSTPlayBin(const URL& url)
30553059
return;
30563060
}
30573061

3062+
#if !RELEASE_LOG_DISABLED
30583063
auto logIdentifier = makeString(hex(reinterpret_cast<uintptr_t>(mediaPlayerLogIdentifier())));
30593064
GST_INFO_OBJECT(m_pipeline.get(), "WebCore logs identifier for this pipeline is: %s", logIdentifier.ascii().data());
3065+
#endif
3066+
30603067
registerActivePipeline(m_pipeline);
30613068

30623069
setStreamVolumeElement(GST_STREAM_VOLUME(m_pipeline.get()));

0 commit comments

Comments
 (0)