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.<div>
<div><br></div><div>-Darin</div><div><br><br><div class="gmail_quote">On Tue, Feb 28, 2012 at 10:47 AM, Simon Fraser <span dir="ltr"><<a href="mailto:simon.fraser@apple.com">simon.fraser@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">"Let's auto generate it" doesn't logically follow from "it's getting large and unwieldy" to me.<div>
<br></div><div>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 somehow.</div><span class="HOEnZb"><font color="#888888"><div>
<br></div></font></span><div><span class="HOEnZb"><font color="#888888">Simon</font></span><div><div class="h5"><div><div><br><div><div>On Feb 28, 2012, at 10:27 AM, Darin Fisher wrote:</div><br><blockquote type="cite">Good idea!<div>
-Darin<br><div><br><div class="gmail_quote">On Tue, Feb 28, 2012 at 8:46 AM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

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