[webkit-reviews] review granted: [Bug 198185] [iOS] Layout Test http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-link-decoration.html is frequently failing or timing out : [Attachment 381141] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 17 02:22:56 PDT 2019


youenn fablet <youennf at gmail.com> has granted Katherine_cheney at apple.com's
request for review:
Bug 198185: [iOS] Layout Test
http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-wi
th-link-decoration.html is frequently failing or timing out
https://bugs.webkit.org/show_bug.cgi?id=198185

Attachment 381141: Patch

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




--- Comment #12 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 381141
  --> https://bugs.webkit.org/attachment.cgi?id=381141
Patch

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

>
LayoutTests/http/tests/resourceLoadStatistics/website-data-removal-for-site-nav
igated-to-with-link-decoration.html:241
> +		  
testRunner.setStatisticsNotifyPagesWhenDataRecordsWereScanned(true);

Is this additional call needed?
The testRunner.installStatisticsDidScanDataRecordsCallback call above shoud set
it to true.


More information about the webkit-reviews mailing list