ProgressEvent could be used in filesystem FileWriter in worker thread, so we should add ProgressEvent in idl file.
Created attachment 143776 [details] Patch
Comment on attachment 143776 [details] Patch Is it possible to write a test for this? It seems like there should be some thing that is noticeably failing.
Created attachment 144316 [details] Patch v2 It's covered by existing test actually.
Comment on attachment 144316 [details] Patch v2 Covered by existing tests, no regressions on any platform according to bots. reasonable change. r+
Comment on attachment 144316 [details] Patch v2 Clearing flags on attachment: 144316 Committed r118698: <http://trac.webkit.org/changeset/118698>
All reviewed patches have been landed. Closing bug.
(In reply to comment #3) > Created an attachment (id=144316) [details] > Patch v2 > > It's covered by existing test actually. Was fast/filesystem/workers/file-writer-events.html failing before this? If so, why wasn't it removed from any skipped files?
(In reply to comment #7) > (In reply to comment #3) > > Created an attachment (id=144316) [details] [details] > > Patch v2 > > > > It's covered by existing test actually. > > Was fast/filesystem/workers/file-writer-events.html failing before this? > > If so, why wasn't it removed from any skipped files? It seems that all platforms(except for chromium) don't support FileSystem API in upstreaming. Chromium is using v8 so it's not affected by this. We are implementing FileSystem API for blackberry porting and found this problem. Blackberry implement of FileSystem API will be upstreaming and you will see we don't skip fast/filesystem.
(In reply to comment #8) > (In reply to comment #7) > > (In reply to comment #3) > > > Created an attachment (id=144316) [details] [details] [details] > > > Patch v2 > > > > > > It's covered by existing test actually. > > > > Was fast/filesystem/workers/file-writer-events.html failing before this? > > > > If so, why wasn't it removed from any skipped files? > > It seems that all platforms(except for chromium) don't support FileSystem API in upstreaming. Chromium is using v8 so it's not affected by this. We are implementing FileSystem API for blackberry porting and found this problem. Blackberry implement of FileSystem API will be upstreaming and you will see we don't skip fast/filesystem. Thanks!