Bug 80189

Summary: Basic IndexedDB shared worker test
Product: WebKit Reporter: David Grogan <dgrogan>
Component: New BugsAssignee: David Grogan <dgrogan>
Status: RESOLVED FIXED    
Severity: Normal CC: jsbell, michaeln, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch for landing
none
fix changelog none

Description David Grogan 2012-03-02 13:41:44 PST
Basic IndexedDB shared worker test
Comment 1 David Grogan 2012-03-02 13:52:09 PST
Created attachment 129953 [details]
Patch
Comment 2 David Grogan 2012-03-02 14:46:12 PST
Created attachment 129959 [details]
Patch
Comment 3 David Grogan 2012-03-02 15:57:45 PST
Created attachment 129974 [details]
Patch
Comment 4 David Grogan 2012-03-02 17:13:46 PST
Tony, could you review this?

Josh and Michael, comments welcome.

This test will be enabled in https://chromiumcodereview.appspot.com/9585031/
Comment 5 Tony Chang 2012-03-05 11:21:04 PST
Comment on attachment 129974 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=129974&action=review

> LayoutTests/fast/js/resources/js-test-pre.js:476
> +        worker.port.onmessage = function(evt) { worker.onmessage(evt); };

Nit: evt -> event

> LayoutTests/storage/indexeddb/resources/shared.js:7
> +    self.postMessage = function(msg)

Nit: msg -> message

> LayoutTests/storage/indexeddb/resources/shared.js:12
> +    }

Nit: semicolon

> LayoutTests/storage/indexeddb/resources/shared.js:24
> +    }

Nit: semicolon
Comment 6 David Grogan 2012-03-05 13:00:30 PST
Created attachment 130187 [details]
Patch for landing
Comment 7 David Grogan 2012-03-05 13:12:55 PST
Created attachment 130191 [details]
fix changelog
Comment 8 WebKit Review Bot 2012-03-05 17:42:16 PST
Comment on attachment 130191 [details]
fix changelog

Clearing flags on attachment: 130191

Committed r109828: <http://trac.webkit.org/changeset/109828>
Comment 9 WebKit Review Bot 2012-03-05 17:42:21 PST
All reviewed patches have been landed.  Closing bug.