<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jan 20, 2009, at 12:55 AM, David Levin wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">As part of bug&nbsp;<a href="https://bugs.webkit.org/show_bug.cgi?id=22720">https://bugs.webkit.org/show_bug.cgi?id=22720</a>, I just added this interface (<a href="https://bugs.webkit.org/attachment.cgi?id=26855&amp;action=diff">https://bugs.webkit.org/attachment.cgi?id=26855&amp;action=diff</a>)&nbsp;with the comments given in this thread largely addressed.<div> <br></div><div>One issue came up. &nbsp;Given that the "loader" will be used by both XHR and importScripts in workers. &nbsp;The name XMLHttpRequestLoader seemed too specific.</div><div><br></div><div>How does ScriptExecutionContextLoader sound? &nbsp;(There will be Document and Worker subclasses of it: DocumentContextLoader and WorkerContextLoader.)</div></blockquote><div><br></div><div>That name doesn't really clarify to me how it differs from and relates to other kinds of loaders. The name itself sounds like the class loads a script execution context, which is clearly not the case from your explanation. Or perhaps you mean that it loads on behalf of a script execution context, but is that more the case for this loader than for any other kind?</div><div><br></div><div>Regards,</div><div>Maciej</div><br><blockquote type="cite"> <div><br></div><div>Thanks,</div><div>Dave<br><div><div><br><br><div class="gmail_quote">On Wed, Dec 31, 2008 at 9:04 AM, Maciej Stachowiak <span dir="ltr">&lt;<a href="mailto:mjs@apple.com">mjs@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 class="Ih2E3d"><br> On Dec 30, 2008, at 12:07 PM, Darin Adler wrote:<br> <br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> I have a few thoughts on this. The general approach seems OK.<br> <br> On Dec 30, 2008, at 11:11 AM, David Levin wrote:<br> <br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> &nbsp; class ScriptResourceLoader {<br> </blockquote> <br> I'm not sure "Script" is the right word here, but I don't have a better one. Up until now Script has meant "interface to the JavaScript interpreter" rather than objects outside that that are intended for use by script. But prefixes like "Programmatic" are uglier, so maybe we should stick with Script. Lets see if we can &nbsp;think of a better prefix.<br> </blockquote> <br></div> ScriptResourceLoader sounds like the name for a ResourceLoader that loads Scripts. However, this is neither a ResourceLoader (in the subclass sense), nor does it load scripts. It is just a helper class to do the loading for XMLHttpRequest. Thus I would suggest the name XHRLoader or XMLHttpRequestLoader.<div class="Ih2E3d"> <br> <br> <br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br> <br> How were you planning on handling synchronous loads? Maybe the function for that should be here too as a static member function?<br> <br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> &nbsp; public:<br> &nbsp; &nbsp; &nbsp; static PassRefPtr&lt;ScriptResourceLoader> create(ScriptExecutionContext*, ScriptResourceLoaderClient*, const ResourceRequest&amp;, bool skipCanLoadCheck, bool sendResourceLoadCallbacks, bool shouldContentSniff);<br> </blockquote> <br> Despite their use in existing code I think booleans are a lousy way to handle options like these.<br> <br> I think we can omit skipCanLoadCheck, since it's always false for XMLHttpRequest. For the other two I would prefer something easier to read at the call site, either named enums or a flags word. The best example I could find of the named enum approach is EChildrenOnly in markup.h, although the use of an "E" prefix isn't desirable. I couldn't find a good example of a flags word.<br> <br> But this may be in appropriate, since we already use booleans for this purpose and you're just refactoring.<br> </blockquote> <br></div> I believe XHR also never does content sniffing, so sendResourceLoadCallbacks would be the only flag that could have an effect. Does it actually differ for different XHR invocations?<br> <br> Regards,<br><font color="#888888"> Maciej<br> <br> </font></blockquote></div><br></div></div></div></blockquote></div><br></body></html>