[webkit-reviews] review granted: [Bug 225613] Speculative revalidation requests not properly attributed as app-bound : [Attachment 428206] Patch

[bugzilla-daemon at webkit.org]

Brent Fulgham <bfulgham at webkit.org> has granted katherine_cheney at apple.com's
request for review:
Bug 225613: Speculative revalidation requests not properly attributed as
app-bound
https://bugs.webkit.org/show_bug.cgi?id=225613

Attachment 428206: Patch

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




--- Comment #5 from Brent Fulgham <bfulgham at webkit.org> ---
Comment on attachment 428206
  --> https://bugs.webkit.org/attachment.cgi?id=428206
Patch

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

r=me

>> LayoutTests/platform/ios-wk2/TestExpectations:1794
>>
+http/tests/in-app-browser-privacy/app-bound-attribution-speculative-revalidati
on.html [ Skip ]
> 
> Why do we need to skip on iOS? Is this because of missing test features, or
does the test just not work on iOS?

I spoke with Kate about this offline. It's skipped in iOS on Open Source. So
this is fine (and will be unskilled in future).