[webkit-reviews] review granted: [Bug 204137] [GTK][WPE] Add same-site cookie support : [Attachment 389666] Patch

[bugzilla-daemon at webkit.org]

Carlos Garcia Campos <cgarcia at igalia.com> has granted Patrick Griffis
<pgriffis at igalia.com>'s request for review:
Bug 204137: [GTK][WPE] Add same-site cookie support
https://bugs.webkit.org/show_bug.cgi?id=204137

Attachment 389666: Patch

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




--- Comment #13 from Carlos Garcia Campos <cgarcia at igalia.com> ---
Comment on attachment 389666
  --> https://bugs.webkit.org/attachment.cgi?id=389666
Patch

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

LGTM

> Source/WebCore/platform/network/HTTPParsers.cpp:988
> +bool isSafeMethod(const String& method)

Add a comment linking to https://tools.ietf.org/html/rfc7231#section-4.2.1