[webkit-reviews] review granted: [Bug 181077] DeferredLoadingScope incorrectly disabled images or enables deferred loading : [Attachment 330005] Fixes the bug

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 20 22:06:24 PST 2017


Wenson Hsieh <wenson_hsieh at apple.com> has granted Ryosuke Niwa
<rniwa at webkit.org>'s request for review:
Bug 181077: DeferredLoadingScope incorrectly disabled images or enables
deferred loading
https://bugs.webkit.org/show_bug.cgi?id=181077

Attachment 330005: Fixes the bug

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




--- Comment #2 from Wenson Hsieh <wenson_hsieh at apple.com> ---
Comment on attachment 330005
  --> https://bugs.webkit.org/attachment.cgi?id=330005
Fixes the bug

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

>
LayoutTests/editing/pasteboard/pasting-with-images-disabled-should-not-enable-d
eferred-loading.html:14
> +    imageLoaded = false;

Let's remove this before landing (...unless it's a special global flag used by
one of the test harness scripts?)

>
LayoutTests/editing/pasteboard/pasting-with-images-disabled-should-not-enable-d
eferred-loading.html:16
> +    setTimeout(() => finishJSTest(), 1000);

Let's remove this too.


More information about the webkit-reviews mailing list