(#1166) - web workers

[calvinmetcalf]

fix in the ajax section and some basic tests for workers

[calvinmetcalf]

this is rebased and ready to merge (or a 👍 )

[calvinmetcalf]

once this gets merged we should prabably publish version 1.1.0. possibly when @daleharvey is done with his Christmas cruise.

[daleharvey]

Looks perfect, thanks a lot 👍

[calvinmetcalf]

merged 8f73143

[hongnk]

I think web worker is important for browser. Especially iOS on older devices (iPad2), whenever I run bulkDocs the UI could freeze for a while.

I attempted to change all "window" references in pouch-nightly.js to "self", and confirmed it worked in a worker thread, database is saved. The problem is that changes listener is not notified. I think the issue could be api.notifyLocalWindows, which update localStorage to notify changes (?), but localStorage is not available in web worker.

[calvinmetcalf]

@hongnk the current up to date build works in a worker and we test for it now, but as indexedDB and websql aren't available there it can only act as a client for the html adapter.

I'm not sure what your trying to say in part 2 of your comment ...

[hongnk]

Hi Calvin, indexedDB and websql both works fine in web worker (maybe only for newer browser). I tested on Chrome 31 and Safari 6.1 and iOS 7. The only changes needed in pouchdb-nightly.js is to change if(window!=undefined) to if (self!=undefined), or window.openDatabase to self.openDatabase etc ... And after this change the script still works as usual in main thread.

However as I said, localStorage is not available in worker and therefore listening to onstorage to notify changes doesn't work.

[calvinmetcalf]

Opened a new issue, I think if we use global, browserify will deal with
window and self changes
On Jan 8, 2014 7:18 PM, "hongnk" notifications@github.com wrote:

Hi Calvin, indexedDB and websql both works fine in web worker (maybe only
for newer browser). I tested on Chrome 31 and Safari 6.1 and iOS 7. The
only changes needed in pouchdb-nightly.js is to change
if(window!=undefined) to if (self!=undefined), or window.openDatabase to
self.openDatabase etc ... And after this change the script still works as
usual in main thread.

However as I said, localStorage is not available in worker and therefore
listening is onstorage to notify changes doesn't work.

—
Reply to this email directly or view it on GitHubhttps://github.com/daleharvey/pouchdb/pull/1169#issuecomment-31891003
.