[webkit-dev] The Care and Feeding of WebCore Modules

[Darin Fisher]

Good idea!
-Darin

On Tue, Feb 28, 2012 at 8:46 AM, Adam Barth <abarth at webkit.org> wrote:

> We haven't done anything about Settings yet, but Setting is also kind
> of growing out of control.  My initial read is that we should try to
> autogenerate Settings (and maybe some/all of the Settings-related
> boilerplate in the WebKit layer) from an "in" file.
>
> Adam
>
>
> On Tue, Feb 28, 2012 at 7:40 AM, Darin Fisher <darin at chromium.org> wrote:
> > Nice.  Is there a plan for modularizing Settings?
> >
> > On Feb 28, 2012 12:30 AM, "Adam Barth" <abarth at webkit.org> wrote:
> >>
> >> I wrote up a short wiki page explaining how the modules system works
> >> and how to use it when building new features:
> >>
> >> https://trac.webkit.org/wiki/Modules
> >>
> >> We've been making good progress refactoring some existing features to
> >> use the system.  This refactoring both improves the hackability of
> >> WebCore by simplifying the core objects (e.g.,
> >> Page/DOMWindow/Document/Navigator) and paves the cowpaths for new code
> >> to avoid bloating these objects.
> >>
> >> In Bug 79663, Alexey asked why we were moving the WebSocket
> >> declaration out of WorkerContext.idl and into Modules/websockets.
> >> Viewed in isolation, I can understand why that change looks somewhat
> >> mysterious.  Hopefully the wiki page above provides some more context
> >> for the change.  In particular, WebSockets fits neatly into the
> >> modules pattern.  We've already removed almost all mentions of
> >> WebSockets from WebCore proper.  Besides one item in
> >> WebCore::Settings, WorkerContext.idl is the last file in WebCore
> >> proper to mention WebSockets.
> >>
> >> Adam
> >> _______________________________________________
> >> webkit-dev mailing list
> >> webkit-dev at lists.webkit.org
> >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20120228/b2632a66/attachment.html>