Change canvasChanged() to take a Optional<FloatRect>
Created attachment 423212 [details] Patch
Created attachment 423216 [details] Patch
Comment on attachment 423216 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423216&action=review > Source/WebCore/html/canvas/CanvasRenderingContext2DBase.h:322 > + FloatRect backingStoreBounds() const { return FloatRect { { }, FloatSize { canvasBase().size() } }; } I think you can omit FloatRect and FloatSize.
Comment on attachment 423216 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423216&action=review > Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp:152 > +void CanvasCaptureMediaStreamTrack::Source::canvasChanged(CanvasBase& canvas, Optional<FloatRect>) Why are we no longer passing is as a const ref? If anything, the object is larger now...
Changed to const Optional<FloatRect>& https://trac.webkit.org/changeset/274436/webkit
<rdar://problem/75445274>