Created attachment 387976 [details]
Patch

Comment on attachment 387976 [details]
Patch

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

r=me, with a few comments

> Websites/webkit.org/wp-content/plugins/social-meta.php:84
> +        if ($image_url[0] === '/')

Should we move this logic to the end of the initial `<?php` block at the top of this file?  That way, if HTML tags are added for other social platforms above this, they won't "miss" this logic.

> Websites/webkit.org/wp-content/plugins/social-meta.php:85
> +            $image_url = get_home_url(null,$image_url);

Style: should have space between `null, $image_url`.

Created attachment 387986 [details]
Patch

Comment on attachment 387986 [details]
Patch

Clearing flags on attachment: 387986

Committed r254731: <https://trac.webkit.org/changeset/254731>

All reviewed patches have been landed.  Closing bug.

<rdar://problem/58665624>