<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">"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><div><br></div><div>Simon<div><div><br><div><div>On Feb 28, 2012, at 10:27 AM, Darin Fisher wrote:</div><br class="Apple-interchange-newline"><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">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 class="HOEnZb"><font color="#888888"><br>
Adam<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Tue, Feb 28, 2012 at 7:40 AM, Darin Fisher <<a href="mailto:darin@chromium.org">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">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">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">webkit-dev@lists.webkit.org</a><br>http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev<br></blockquote></div><br></div></div></div></body></html>