Bug 195211 – [WPE] Public API headers are missing autocleanup definitions

• Status: RESOLVED
• Resolution: FIXED
• Component: WPE WebKit

[Adrian Perez]

For the GTK port, there are the following headers which define them:

  * Source/WebKit/UIProcess/API/gtk/WebKitAutocleanups.h
  * Source/WebKit/WebProcess/InjectedBundle/API/gtk/WebKitWebExtensionAutocleanups.h

…but the WPE port does not have them.

[Adrian Perez]

Created attachment 363336 [details]
Patch

[Adrian Perez]

Created attachment 363341 [details]
Patch v2

[Adrian Perez]

If there are no objections, I would like to have this merged
to the 2.24 release branch as well =)

[Philippe Normand]

Comment on attachment 363341 [details]
Patch v2

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

> Source/WebKit/UIProcess/API/wpe/WebKitAutocleanups.h:13
> + * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''

Any reason for using Apple's copyright header instead of the usual LGPL header?

[Adrian Perez]

(In reply to Philippe Normand from comment #4)
> Comment on attachment 363341 [details]
> Patch v2
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=363341&action=review
> 
> > Source/WebKit/UIProcess/API/wpe/WebKitAutocleanups.h:13
> > + * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
> 
> Any reason for using Apple's copyright header instead of the usual LGPL
> header?

I guess none. In this case the headers were copied from the GTK port
and adapted, so the GTK port headers are also using this instead of the
LGPL license blurb. My guess is that when the headers for the autocleanups
were first added for the GTK port nobody realized during the review that
the LGPL version should have been used (as seen in the rest of GTK/WPE
API headers).

[Philippe Normand]

Comment on attachment 363341 [details]
Patch v2

Let's land this and fix the headers later on.

[Adrian Perez]

(In reply to Adrian Perez from comment #5)
> (In reply to Philippe Normand from comment #4)
> > Comment on attachment 363341 [details]
> > Patch v2
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=363341&action=review
> > 
> > > Source/WebKit/UIProcess/API/wpe/WebKitAutocleanups.h:13
> > > + * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
> > 
> > Any reason for using Apple's copyright header instead of the usual LGPL
> > header?
> 
> I guess none. In this case the headers were copied from the GTK port
> and adapted, so the GTK port headers are also using this instead of the
> LGPL license blurb. My guess is that when the headers for the autocleanups
> were first added for the GTK port nobody realized during the review that
> the LGPL version should have been used (as seen in the rest of GTK/WPE
> API headers).

There's only a few authors for these files, so it should be easy to
find the people and have them agree to change the license header. I have
filed bug #195245 to do that.

[WebKit Commit Bot]

Comment on attachment 363341 [details]
Patch v2

Clearing flags on attachment: 363341

Committed r242312: <https://trac.webkit.org/changeset/242312>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.