[webkit-reviews] review granted: [Bug 219493] [GStreamer] Fix video losing size at the end of the stream : [Attachment 415305] Patch

[bugzilla-daemon at webkit.org]

Xabier Rodríguez Calvar <calvaris at igalia.com> has granted Alicia Boya García
<aboya at igalia.com>'s request for review:
Bug 219493: [GStreamer] Fix video losing size at the end of the stream
https://bugs.webkit.org/show_bug.cgi?id=219493

Attachment 415305: Patch

https://bugs.webkit.org/attachment.cgi?id=415305&action=review




--- Comment #5 from Xabier Rodríguez Calvar <calvaris at igalia.com> ---
Comment on attachment 415305
  --> https://bugs.webkit.org/attachment.cgi?id=415305
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=415305&action=review

>
Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:1066
> +		       existingTrack->setPad(GRefPtr(pad));

Nit: Unless that for some reason you want to keep it explicit, you can remove
the GRefPtr constructor that will be called implicitly.

>
Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:1147
> +		       existingTrack->setPad(GRefPtr(pad));

Ditto.

> Source/WebCore/platform/graphics/gstreamer/TrackPrivateBaseGStreamer.cpp:179
> +void TrackPrivateBaseGStreamer::setPad(GRefPtr<GstPad>&& pad)
> +{
> +    m_pad = WTFMove(pad);
> +}

Nit: I think you can inline this in the header file.