[webkit-changes] [WebKit/WebKit] b842b3: [WK2][GLib] Clean up GLib ArgumentCoders specializ...

Žan Doberšek noreply at github.com
Mon Nov 7 01:28:08 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b842b3761eaf73ad3c9d28ad08c34c81e77b0b94
      https://github.com/WebKit/WebKit/commit/b842b3761eaf73ad3c9d28ad08c34c81e77b0b94
  Author: Žan Doberšek <zdobersek at igalia.com>
  Date:   2022-11-07 (Mon, 07 Nov 2022)

  Changed paths:
    M Source/WebKit/Shared/glib/ArgumentCodersGLib.cpp
    M Source/WebKit/Shared/glib/ArgumentCodersGLib.h

  Log Message:
  -----------
  [WK2][GLib] Clean up GLib ArgumentCoders specializations
https://bugs.webkit.org/show_bug.cgi?id=247561

Reviewed by Carlos Garcia Campos.

In ArgumentCoder specializations for GVariant and GTlsCertificate objects
managed through GRefPtrs, have the encode methods accept simple references to
the GRefPtrs, avoiding unnecessary refcounting churn.

Drop the encoder and decoder type templates on the GTlsCertificate
specialization of ArgumentCoder and subsequently-necessary specializations
for the IPC::Encoder and IPC::Decoder methods. We can enforce those two types
without the template, and support for other encoders and decoders likely won't
be necessary.

* Source/WebKit/Shared/glib/ArgumentCodersGLib.cpp:
(IPC::ArgumentCoder<GRefPtr<GVariant>>::encode):
(IPC::ArgumentCoder<GRefPtr<GTlsCertificate>>::encode):
(IPC::ArgumentCoder<GRefPtr<GTlsCertificate>>::decode):
* Source/WebKit/Shared/glib/ArgumentCodersGLib.h:

Canonical link: https://commits.webkit.org/256389@main




More information about the webkit-changes mailing list