Title: [231759] trunk/Source/WebCore
- Revision
- 231759
- Author
- commit-qu...@webkit.org
- Date
- 2018-05-14 09:18:36 -0700 (Mon, 14 May 2018)
Log Message
[GStreamer] Fix style issue in MediaPlayerPrivateGStreamerBase
https://bugs.webkit.org/show_bug.cgi?id=185510
Patch by Thibault Saunier <tsaun...@igalia.com> on 2018-05-14
Reviewed by Philippe Normand.
ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:629: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:684: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:807: More than one command on the same line [whitespace/newline] [4]
Indentation and style issue fixed only.
* platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::volumeChangedCallback):
(WebCore::MediaPlayerPrivateGStreamerBase::muteChangedCallback):
(WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (231758 => 231759)
--- trunk/Source/WebCore/ChangeLog 2018-05-14 15:59:56 UTC (rev 231758)
+++ trunk/Source/WebCore/ChangeLog 2018-05-14 16:18:36 UTC (rev 231759)
@@ -1,3 +1,21 @@
+2018-05-14 Thibault Saunier <tsaun...@igalia.com>
+
+ [GStreamer] Fix style issue in MediaPlayerPrivateGStreamerBase
+ https://bugs.webkit.org/show_bug.cgi?id=185510
+
+ Reviewed by Philippe Normand.
+
+ ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:629: More than one command on the same line [whitespace/newline] [4]
+ ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:684: More than one command on the same line [whitespace/newline] [4]
+ ERROR: Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:807: More than one command on the same line [whitespace/newline] [4]
+
+ Indentation and style issue fixed only.
+
+ * platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
+ (WebCore::MediaPlayerPrivateGStreamerBase::volumeChangedCallback):
+ (WebCore::MediaPlayerPrivateGStreamerBase::muteChangedCallback):
+ (WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):
+
2018-05-14 Zalan Bujtas <za...@apple.com>
[LFC] Implement height computation for non-replaced out of flow elements.
Modified: trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp (231758 => 231759)
--- trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp 2018-05-14 15:59:56 UTC (rev 231758)
+++ trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp 2018-05-14 16:18:36 UTC (rev 231759)
@@ -615,7 +615,9 @@
// This is called when m_volumeElement receives the notify::volume signal.
GST_DEBUG_OBJECT(player->pipeline(), "Volume changed to: %f", player->volume());
- player->m_notifier->notify(MainThreadNotification::VolumeChanged, [player] { player->notifyPlayerOfVolumeChange(); });
+ player->m_notifier->notify(MainThreadNotification::VolumeChanged, [player] {
+ player->notifyPlayerOfVolumeChange();
+ });
}
MediaPlayer::NetworkState MediaPlayerPrivateGStreamerBase::networkState() const
@@ -670,7 +672,9 @@
void MediaPlayerPrivateGStreamerBase::muteChangedCallback(MediaPlayerPrivateGStreamerBase* player)
{
// This is called when m_volumeElement receives the notify::mute signal.
- player->m_notifier->notify(MainThreadNotification::MuteChanged, [player] { player->notifyPlayerOfMute(); });
+ player->m_notifier->notify(MainThreadNotification::MuteChanged, [player] {
+ player->notifyPlayerOfMute();
+ });
}
void MediaPlayerPrivateGStreamerBase::acceleratedRenderingStateChanged()
@@ -793,7 +797,9 @@
if (triggerResize) {
GST_DEBUG_OBJECT(pipeline(), "First sample reached the sink, triggering video dimensions update");
- m_notifier->notify(MainThreadNotification::SizeChanged, [this] { m_player->sizeChanged(); });
+ m_notifier->notify(MainThreadNotification::SizeChanged, [this] {
+ m_player->sizeChanged();
+ });
}
if (!m_renderingCanBeAccelerated) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
