Bug 166460 – Relicense non-distributable web inspector images

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Michael Catanzaro]

The file Source/WebInspectorImages/APPLE_IMAGES_LICENSE.rtf begins with the following text:

"""
Apple Inc.
Software License Agreement for Web Inspector Images
Internal Use License Only – No Redistribution Permitted

PLEASE READ THIS SOFTWARE LICENSE AGREEMENT ("LICENSE") CAREFULLY BEFORE USING THE APPLE SOFTWARE INCLUDED IN THIS PACKAGE.  BY USING THIS SOFTWARE, YOU ARE AGREEING TO BE BOUND BY THE TERMS OF THIS LICENSE.  IF YOU DO NOT AGREE TO THE TERMS OF THIS LICENSE, DO NOT USE THIS SOFTWARE.
"""

Clearly this is not acceptable content for the WebKit open source project. It serves no purpose in OpenSource except as a trap to other WebKit ports (see bug #136632, bug #166459) and downstream projects, as it is not usable except by Apple. It should move to Apple's internal repositories until such time that Apple decides to relicense the images under an acceptable content license, e.g. one of the licenses listed at [1]. In the meantime, the GTK port's CC-BY-SA inspector images should be used on all ports.

Note that there are fewer GTK images than Apple images. The following Apple icons have no GTK replacement:

ActiveCallFrame.svg, AnimationPlayStatePaused.svg, AnimationPlayStateRunning.svg, CallTrees.svg, Camera.svg, ClippingCSSLarge.png, [e-mail], ClippingGenericLarge.png, [e-mail], ClippingJSLarge.png, [e-mail], Compare.svg,DocumentCSSLarge.png, [e-mail], DocumentFontLarge.png, [e-mail], DocumentGenericLarge.png, [e-mail], DocumentImageLarge.png, [e-mail], DocumentJSLarge.png, [e-mail], DocumentMarkupLarge.png, [e-mail], Events.svg, HeapAllocationsInstrument.svg, HeapSnapshot.svg, HeapSnapshotDiff.svg, HeapSnapshotInstances.svg, HeapSnapshotSelected.svg, HeapSnapshotSummary.svg, LayoutInstrument.svg, ListStylePositionInside.svg, ListStylePositionOutside.svg, MemoryInsurment.svg, NetworkInstrument.svg, RenderingFramesInstrument.svg, ScriptsInstrument.svg, Search.svg, Stopwatch.svg, TimelineRecordGarbageCollection.svg, WorkerScript.png, [e-mail], WorkerScriptLarge.png, [e-mail]

Note that the GTK images really only exist due to this legal issue. If the Apple images were appropriately-licensed, we could get rid of ours. Although it might seem nice to have images that fit better into GTK, it's not really, because the entire web inspector UI is clearly designed to look like an Apple product, not a GTK+ application.

[1] https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing

[Michael Catanzaro]

Oops, forgot the patch. :P

[Michael Catanzaro]

Created attachment 297711 [details]
Patch

[Blaze Burg]

We are re-investigating what we are able to do w.r.t. licensing.

In the meantime, I am not opposed to this in principle, but we are unable to make necessary changes internally until folks are back from holiday break. So please defer this patch until we find a way to make this work without breaking the Mac build.

[Radar WebKit Bug Importer]

<rdar://problem/29800966>

[Michael Catanzaro]

(In reply to comment #3)
> We are re-investigating what we are able to do w.r.t. licensing.

Great, thanks!

> In the meantime, I am not opposed to this in principle, but we are unable to
> make necessary changes internally until folks are back from holiday break.
> So please defer this patch until we find a way to make this work without
> breaking the Mac build.

Yeah, definitely not planning to land this during the holidays. But I figured having a patch would help... expedite a resolution to this issue. ;)

I don't think it breaks the Mac build, though; EWS is clearly green. You will just have a few missing images, the same ones we're missing currently.

[Michael Catanzaro]

Hey Brian et. al., I think we should proceed with an updated version of this patch until such time that you're able to resolve the licensing issues. These files don't belong in OpenSource.

[Joseph Pecoraro]

Created attachment 326930 [details]
[PATCH] Proposed Fix

We've gotten the approval we need within Apple to remove the restrictive license on Web Inspector images.

[Blaze Burg]

Comment on attachment 326930 [details]
[PATCH] Proposed Fix

r=me

[Timothy Hatcher]

Awesome!

[Michael Catanzaro]

Thanks a bunch!

And thanks also to Fujii, for helping us keep up with new GTK images recently. Fujii, any objections to removing the GTK inspector images now that we can use Apple's? Seems like it's probably best to unify on one set of images now that we're able to do so?

[WebKit Commit Bot]

Comment on attachment 326930 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 326930

Committed r224861: <https://trac.webkit.org/changeset/224861>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Konstantin Tokarev]

Awesome!

[Fujii Hironori]

Wow, Amazing! I have no objection to remove GTK's images.

[Andres Gomez Garcia]

Awesome!

Thanks to all the people involved and specially to the people at Apple who have pushed and succeeded on getting the approval for relicensing this :)

Thanks a lot!

[Carlos Garcia Campos]

Yeeeeeees! Thank you guys.