[webkit-dev] The Care and Feeding of WebCore Modules
darin at chromium.org
Tue Feb 28 11:11:42 PST 2012
The main issue isn't that the settings are large and unwieldy. I thought
the point of the modularization effort was to enable partitioning of
features. That means eliminating files that enumerate each feature. That
said, we might still have classes that need to mention all features, so to
address that you might auto-gen such classes.
On Tue, Feb 28, 2012 at 10:47 AM, Simon Fraser <simon.fraser at apple.com>wrote:
> "Let's auto generate it" doesn't logically follow from "it's getting large
> and unwieldy" to me.
> It seems that a better approach would be to figure out how to simplify
> Settings (do we still need them all?), and if we do, perhaps to break it up
> On Feb 28, 2012, at 10:27 AM, Darin Fisher wrote:
> Good idea!
> 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.
>> 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
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the webkit-dev