[webkit-reviews] review granted: [Bug 225254] [Cocoa] Calling into -[AVPlayerItem currentTime] is very expensive : [Attachment 427458] Patch

[bugzilla-daemon at webkit.org]

Eric Carlson <eric.carlson at apple.com> has granted Jer Noble
<jer.noble at apple.com>'s request for review:
Bug 225254: [Cocoa] Calling into -[AVPlayerItem currentTime] is very expensive
https://bugs.webkit.org/show_bug.cgi?id=225254

Attachment 427458: Patch

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




--- Comment #3 from Eric Carlson <eric.carlson at apple.com> ---
Comment on attachment 427458
  --> https://bugs.webkit.org/attachment.cgi?id=427458
Patch

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

>
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundati
onObjC.mm:1109
> +    auto currentTimeInterval = CMTimeMake(1, 10);

Do we need this local variable?