We should have at least one that verifies basic functionality. There shouldn't be any difference in implementation between shared and dedicated workers, so one's probably plenty.
Created attachment 136993 [details] Patch
Comment on attachment 136993 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=136993&action=review > LayoutTests/fast/filesystem/workers/file-writer-events-shared.html:14 > + //worker.port.postMessage("IMPORT:../../filesystem/simple-persistent.html"); Do we need these commented out lines? > LayoutTests/fast/js/resources/js-test-pre.js:509 > +var workerPort; Probably we should put this var together with other global var declarations at the beginning of this file.
(In reply to comment #2) > (From update of attachment 136993 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=136993&action=review > > > LayoutTests/fast/filesystem/workers/file-writer-events-shared.html:14 > > + //worker.port.postMessage("IMPORT:../../filesystem/simple-persistent.html"); > > Do we need these commented out lines? Nope--accidentally left them in there while experimenting. Removed. > > LayoutTests/fast/js/resources/js-test-pre.js:509 > > +var workerPort; > > Probably we should put this var together with other global var declarations at the beginning of this file. Actually, it doesn't need to be global at all, due to the closures. Changed to local.
Created attachment 138388 [details] Patch
Ping? This is quite small, and fixes a piece of the test harness that misrepresents itself as supporting shared workers.
I suspect Levin could review this tiny patch, even though he's a busy-busy man these days.
Comment on attachment 138388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=138388&action=review > LayoutTests/fast/filesystem/workers/file-writer-events-shared.html:1 > +<!DOCTYPE HTML> Personally I would prefer -shared-work instead of just -shared (because I can't figure out what -shared means immediately when I see it like this.). But I'm not so adamant that I think you must do this before checking in. Consider this a "If it isn't too much trouble, it sure would be nice if..."
Comment on attachment 138388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=138388&action=review >> LayoutTests/fast/filesystem/workers/file-writer-events-shared.html:1 >> +<!DOCTYPE HTML> > > Personally I would prefer -shared-work instead of just -shared (because I can't figure out what -shared means immediately when I see it like this.). > > But I'm not so adamant that I think you must do this before checking in. > > Consider this a "If it isn't too much trouble, it sure would be nice if..." Switched to -shared-worker.
Committed r118246: <http://trac.webkit.org/changeset/118246>