[webkit-reviews] review granted: [Bug 177442] [EME] Add ClearKey support for persistent session data load and removal : [Attachment 321706] Patch

[bugzilla-daemon at webkit.org]

Xabier Rodríguez Calvar <calvaris at igalia.com> has granted Zan Dobersek
<zan at falconsigh.net>'s request for review:
Bug 177442: [EME] Add ClearKey support for persistent session data load and
removal
https://bugs.webkit.org/show_bug.cgi?id=177442

Attachment 321706: Patch

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




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

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

> Source/WebCore/platform/encryptedmedia/clearkey/CDMClearKey.cpp:458
> +    // Use a helper functor that schedules the callback dispatch, avoiding
> +    // duplicated callOnMainThread() calls.

You can have this in a single line.

> Source/WebCore/platform/encryptedmedia/clearkey/CDMClearKey.cpp:503
> +    // Use a helper functor that schedules the callback dispatch, avoiding
> +    // duplicated callOnMainThread() calls.

You can have this in a single line.