<div class="gmail_quote">On Wed, Aug 26, 2009 at 12:49 AM, Darin Fisher <span dir="ltr"><<a href="mailto:darin@google.com">darin@google.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="gmail_quote"><div class="im">On Sun, Aug 23, 2009 at 11:33 PM, Robert O'Callahan <span dir="ltr"><<a href="mailto:robert@ocallahan.org" target="_blank">robert@ocallahan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Sat, Aug 22, 2009 at 10:22 PM, Jeremy Orlow <span dir="ltr"><<a href="mailto:jorlow@chromium.org" target="_blank">jorlow@chromium.org</a>></span> wrote:<br></div><div class="gmail_quote"><div>
<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><br></blockquote></div></div></blockquote></div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><div class="gmail_quote"><div></div><div>

<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">




</blockquote></div></div></blockquote><div class="gmail_quote"><div>But getStorageUpdates is still not the right name for it.  The only way that there'd be any updates to get is if, when you call the function, someone else takes the lock and makes some updates.  Maybe it should be yieldStorage (or yieldStorageMutex)?  In other words, maybe the name should imply that you're allowing concurrent updates to happen?</div>


</div></div></div></blockquote><div class="gmail_quote">

</div>
</div></div><br>I thought that's what getStorageUpdates implied :-).<div><div></div><div><br clear="all"></div></div></blockquote><div><br></div></div><div>The getStorageUpdates name seems pretty decent to me when considering it from the perspective of the caller.  The caller is saying that they are OK with being able to see changes made to the localStorage by "other threads."  This cleverly avoids the need to talk about locks, which seems like a good thing.  It is okay for there to be no updates to storage.</div>

<div><br></div><font color="#888888"><div>-Darin</div></font></div>
</blockquote></div><br><div><br></div><div>What about allowStorageUpdates?</div><div><br></div><div>-Darin</div>