Re: [DataCache] Some Corrections

2009-12-17 Thread Nikunj Mehta
Joseph Pecoraro wrote: I have changed to using the new method "immediate" and that also removed this call. Immediate looks useful. The specification for immediate is: [[ When this method is called, the user agent creates a new cache transaction, and performs the steps to

Re: [DataCache] Some Corrections

2009-12-16 Thread Nikunj R. Mehta
On Dec 4, 2009, at 9:41 AM, Joseph Pecoraro wrote: - 4.1.1 Examples http://dev.w3.org/2006/webapi/DataCache/#examples Last code example is improperly indented. Fixed - 4.2.1 Opening a Cache has a typo in Step 3.2 http://dev.w3.org/2006/webapi/DataCache/#opening-cache If data cache group

Re: [DataCache] Some Corrections

2009-12-16 Thread Nikunj R. Mehta
On Dec 7, 2009, at 12:01 PM, Joseph Pecoraro wrote: Some more DataCache API Corrections: - 4.1.1. Examples http://dev.w3.org/2006/webapi/DataCache/#examples No such thing as finish() used in the first example. Perhaps this should be commit()? [[ cache.transaction(function(txn) {

Re: [DataCache] Some Corrections

2009-12-16 Thread Joseph Pecoraro
- 4.2.2.1. Starting a transaction - Rules should be switched. http://dev.w3.org/2006/webapi/DataCache/#starting-a-transaction first arm: (these should be swapped) 3.3 - Mark transaction as off-line. 3.4 - Create a new cache transaction called transaction and set data cache to be its

Re: [DataCache] Some Corrections

2009-12-16 Thread Nikunj R. Mehta
On Dec 16, 2009, at 4:09 PM, Joseph Pecoraro wrote: - 4.2.2.1. Starting a transaction - Rules should be switched. http://dev.w3.org/2006/webapi/DataCache/#starting-a-transaction first arm: (these should be swapped) 3.3 - Mark transaction as off-line. 3.4 - Create a new cache transaction

Re: [DataCache] Some Corrections

2009-12-16 Thread Joseph Pecoraro
- 4.2.7. Transaction API http://dev.w3.org/2006/webapi/DataCache/#transaction-interface Extra whitespace for the oncommitted event. [[ IDL interface CacheTransaction { void getItem(in DOMString uri, ItemCallback callback); void release(in DOMString uri); void commit(); //

Re: [DataCache] Some Corrections

2009-12-16 Thread Joseph Pecoraro
Great work on the updates! I have a few new corrections that I hadn't yet submitted and still seem relevant in the new draft. - 4.3.1 Local Server API http://dev.w3.org/2006/webapi/DataCache/#request-interface Make MutableHttpResponse extend HttpResponse? It has methods, but no attributes to

Re: [DataCache] Some Corrections

2009-12-16 Thread Joseph Pecoraro
I have changed to using the new method immediate and that also removed this call. Immediate looks useful. The specification for immediate is: [[ When this method is called, the user agent creates a new cache transaction, and performs the steps to add a resource to be captured in that cache

Re: [DataCache] Some Corrections

2009-12-07 Thread Joseph Pecoraro
Some more DataCache API Corrections: - 4.1.1. Examples http://dev.w3.org/2006/webapi/DataCache/#examples No such thing as finish() used in the first example. Perhaps this should be commit()? [[ cache.transaction(function(txn) { txn.capture(uri); txn.finish(); }); ]] - 4.2.2.2.

[DataCache] Some Corrections

2009-12-04 Thread Joseph Pecoraro
- 4.1.1 Examples http://dev.w3.org/2006/webapi/DataCache/#examples Last code example is improperly indented. - 4.2.1 Opening a Cache has a typo in Step 3.2 http://dev.w3.org/2006/webapi/DataCache/#opening-cache If data cache group does not exist, the perform the following substeps: should be: