Use the async version of WebFileSystem::createFileWriter

Use the async version of WebFileSystem::createFileWriter

Removing unnecessary callback chain.

(Some worker code still rely on the old createFileWriter, but
they're supposed to go away)

This depends on chrome-side patch:
https://codereview.chromium.org/18427005/

BUG=257349
TEST=no behavioral changes

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=154363

[kinuko, 2013-07-12 05:10:06]

PTL

[abarth-chromium, 2013-07-12 22:53:48]

LGTM.  The memory management in this CL is kind of scary to me, but ok.

https://codereview.chromium.org/18411005/diff/5001/Source/web/AsyncFileSystem...
File Source/web/AsyncFileSystemChromium.cpp (right):

https://codereview.chromium.org/18411005/diff/5001/Source/web/AsyncFileSystem...
Source/web/AsyncFileSystemChromium.cpp:117:
m_webFileSystem->createFileWriter(path, writerClient, new
WebKit::WebFileSystemCallbacksImpl(callbacks, asyncFileWriter.release()));
scary "naked" new.  We need to fix that about the API.  Not something to worry
about in this CL.

[kinuko, 2013-07-16 02:35:22]

Thanks!

https://codereview.chromium.org/18411005/diff/5001/Source/web/AsyncFileSystem...
File Source/web/AsyncFileSystemChromium.cpp (right):

https://codereview.chromium.org/18411005/diff/5001/Source/web/AsyncFileSystem...
Source/web/AsyncFileSystemChromium.cpp:117:
m_webFileSystem->createFileWriter(path, writerClient, new
WebKit::WebFileSystemCallbacksImpl(callbacks, asyncFileWriter.release()));
On 2013/07/12 22:53:48, abarth wrote:
> scary "naked" new.  We need to fix that about the API.  Not something to worry
> about in this CL.

Yep... this needs API fix, I'd like to address these memory handling issues in a
separate CL.

[commit-bot: I haz the power, 2013-07-16 10:21:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/18411005/12001

[commit-bot: I haz the power, 2013-07-16 12:33:00]

Retried try job too often on linux_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layo...

[commit-bot: I haz the power, 2013-07-17 04:03:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/18411005/12001

[commit-bot: I haz the power, 2013-07-17 04:32:13]

Change committed as 154363