[webkit-dev] Cleaning House

[Žan Doberšek]

GoogleURL seems Chromium-specific, i.e. WTF_USE_GOOGLEURL is only defined
for Chromium in Source/WebCore/config.h.

Regards,
-Z


On Thu, Apr 4, 2013 at 10:50 AM, Maciej Stachowiak <mjs at apple.com> wrote:

>
> On Apr 4, 2013, at 1:39 AM, Allan Sandfeld Jensen <kde at carewolf.com>
> wrote:
>
> On Thursday 04 April 2013, Geoffrey Garen wrote:
>
> Hi folks.
>
> Since we no longer need to support the Chromium port, let's take the
> opportunity to streamline. Hopefully, this will make development easier
> and more coherent for everyone. Adam and Eric offered to do some of this
> cleanup, but I think it's healthier for people who will continue to be a
> part of WebKit to decide what gets cleaned up, and execute on the plan.
>
> Below is a high-level view of some improvements we're planning over the
> coming weeks.
>
> Concepts we plan to remove:
> Layering violations in WebCore/platform, where a Page* or Frame* is
>
> passed
>
> to a function Supplementable and Supplement
> #if USE(GOOGLEURL)
> #if USE(V8)
> #if !USE(JSC)
> #if PLATFORM(CHROMIUM)
> Skia
> DOMFileSystem
> WebLayer and its scrolling implementation
> Features #defines that haven't gained traction
>
> Unless you plan to scare more ports away, I would suggest double checking
> stuff to remove. As far as I know BlackBerry and EFL are using Skia, and I
> am
> sure there are ports using GoogleURL as well.
>
> For the record though I don't think Qt is using any of that those.
>
>
> Geoff posted the list in part because we'd like to know if any of the
> things above are used by other ports. We're not planning to remove things
> still in use.
>
> It would be good to know specifics. I could not find evidence of other
> ports using GoogleURL for instance.
>
> Regards,
> Maciej
>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20130404/97088fc0/attachment.html>