It's actually a singleton, but the API suggests you can create your own or use the default one, but the default one should be the only one.
Created attachment 256960 [details]
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment on attachment 256960 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=256960&action=review
> + PasteboardHelper::singleton() and make it non-copiable, fast
> + gMarkupPrefix = String::fromUTF8("<meta http-equiv=\"content-type\" content=\"text/html; charset=utf-8\">");
Not terribly important since it's only called once, but this string seems ASCII-only, so you could initialize gMarkipPrefix from an ASCIILiteral.
Also, can this be initialized where it's declared?
> + WTF_MAKE_NONCOPYABLE(PasteboardHelper); WTF_MAKE_FAST_ALLOCATED;
No need for WTF_MAKE_FAST_ALLOCATED. There's only one object alive during the program's lifetime, and that one is constructed in memory that's reserved by the static NeverDestroyed<PasteboardHelper> object.
Created attachment 256962 [details]
Committed r186954: <http://trac.webkit.org/changeset/186954>