Bug 231569 – MediaCapabilities should enqueue a task to resolve promises

• Status: RESOLVED
• Resolution: FIXED
• Component: WebRTC

[youenn fablet]

MediaCapabilities should enqueue a task to resolve promises

[youenn fablet]

Created attachment 440918 [details]
Patch

[Eric Carlson]

Comment on attachment 440918 [details]
Patch

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

> Source/WebCore/Modules/mediacapabilities/MediaCapabilities.cpp:215
>              logger->info(LogMedia, identifier, "::callback() - Resolved. info: ", info);

It looks like this indentation is wrong now

> Source/WebCore/Modules/mediacapabilities/MediaCapabilities.cpp:269
> +            logger->info(LogMedia, identifier, "::callback() - Resolved. info: ", info);

Ditto

[youenn fablet]

Created attachment 441201 [details]
Patch for landing

[youenn fablet]

(In reply to Eric Carlson from comment #2)
> Comment on attachment 440918 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=440918&action=review
> 
> > Source/WebCore/Modules/mediacapabilities/MediaCapabilities.cpp:215
> >              logger->info(LogMedia, identifier, "::callback() - Resolved. info: ", info);
> 
> It looks like this indentation is wrong now
> 
> > Source/WebCore/Modules/mediacapabilities/MediaCapabilities.cpp:269
> > +            logger->info(LogMedia, identifier, "::callback() - Resolved. info: ", info);
> 
> Ditto

Fixed in landing patch

[EWS]

Committed r284236 (243045@main): <https://commits.webkit.org/243045@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 441201 [details].

[Radar WebKit Bug Importer]

<rdar://problem/84290404>