[webkit-reviews] review granted: [Bug 203021] [Clipboard API] Support navigator.clipboard.read() : [Attachment 381222] Patch

[bugzilla-daemon at webkit.org]

Ryosuke Niwa <rniwa at webkit.org> has granted Wenson Hsieh
<wenson_hsieh at apple.com>'s request for review:
Bug 203021: [Clipboard API] Support navigator.clipboard.read()
https://bugs.webkit.org/show_bug.cgi?id=203021

Attachment 381222: Patch

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




--- Comment #2 from Ryosuke Niwa <rniwa at webkit.org> ---
Comment on attachment 381222
  --> https://bugs.webkit.org/attachment.cgi?id=381222
Patch

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

> Source/WebCore/Modules/async-clipboard/Clipboard.cpp:112
> +	       if (itemInfo.changeCount != changeCountAtStart) {

This change count check should really be happening in the UI process side but
it's okay for now.