<br><br><div class="gmail_quote">On Mon, Oct 5, 2009 at 7:50 PM, Drew Wilson <span dir="ltr">&lt;<a href="mailto:atwilson@google.com">atwilson@google.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Ooops, meant to reply to all.<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Mon, Oct 5, 2009 at 7:49 PM, Drew Wilson <span dir="ltr">&lt;<a href="mailto:atwilson@google.com" target="_blank">atwilson@google.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div>On Mon, Oct 5, 2009 at 6:40 PM, Sam Weinig <span dir="ltr">&lt;<a href="mailto:sam.weinig@gmail.com" target="_blank">sam.weinig@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote">
</div></blockquote><div><br></div></div><div>That is not true, they are also available in nightly builds at <a href="http://nightly.webkit.org/" target="_blank">http://nightly.webkit.org/</a>.</div></div></blockquote><div>


<br></div></div><div>I&#39;m not sure what you mean, exactly - the nightly webkit builds all have SharedWorkers turned on, with no way to turn them off short of editing the source code and recompiling (since the only existing implementation of SharedWorkerRepository.isAvailable() returns true). I suspect I&#39;m missing something obvious, though - can you elaborate?</div>


<div><br></div><div>My expectation was that only when I write the Chromium implementation of SharedWorkerRepository will isAvailable() ever return false - so this only affects Chromium deployments.</div></div></blockquote>
</div></div></div></blockquote><div><br></div><div>Ok, but this is not the case for WebSockets.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div>
<div class="h5"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">
<div><br></div><div>Regardless, I don&#39;t think we should rush out to roll all of those features out of the tree, and I certainly don&#39;t think we should be singling out SharedWorkers or WebSockets</div></span></div>





</blockquote><div><br></div></div><div>I don&#39;t mean to single out SharedWorkers or WebSockets, but I don&#39;t see any others using the same technique (barring window.Audio, which I don&#39;t think is the same thing, but should non-the less be fixed).  But, as we have many developers using the nightlies, I think this should be handled with some speed.</div>




</div></blockquote><div><br></div></div><div>Take a look at DOMWindow.cpp - there&#39;s quite a bit of code that does something like &quot;look at settings to see if feature is enabled, return null if not&quot; (DOMWindow::openDatabase(), for an example).</div>




<div><br></div></div></blockquote><div><br></div></div><div>This is indeed unfortunate, but also is a step removed, since it does not really effect feature detection, it is also not a shipping configuration.</div></div></blockquote>


<div><br></div></div><div>Why doesn&#39;t it affect feature detection? Someone can do &quot;if localStorage in window&quot;, yet have their code fail when window.localStorage is null?</div></div></blockquote></div></div>
</div></blockquote><div><br></div><div>I only looked at window.openDatabase(), since that is what you called out.  Again, this is unfortunate and should be rectified, but it does not represent a real concern in my mind, since I don&#39;t know of any shipping webkit based browsers that ship with this configuration on or an accessible preference to get this way.</div>
<div><br></div><div>The good news is that it looks like we can fix this for nightlies by enabling the WebSocket runtime switch.</div><div><br></div><div>I would still very much like a good solution to runtime enabling/disabling features in the bindings as I think this would be a useful addition to the webkit arsenal and I am curious why it is thought that doing it right will be prohibitively expensive (it may very well be, I just don&#39;t know why that is).  </div>
<div><br></div><div>-Sam</div></div>