Split FileAPI code for common|common_child|renderer

Split FileAPI code for common|common_child|renderer

Move following files from content/common to content/common_child:
- webfilesystem_callback_adapters.*
- webfilesystem_impl.*
- webfilewriter_impl.*

Move following files from webkit/fileapi to webkit/common/fileapi:
- directory_entry.h
- file_system_types.h
- file_system_url*
- file_system_util*

Move following files from webkit/fileapi to webkit/renderer/fileapi:
- webfilewriter_base*

BUG=239710
R=jamesr,michaeln 
TBR=avi,thestig

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202501

[michaeln, 2013-05-24 20:41:58]

https://codereview.chromium.org/15658004/diff/13001/content/content_common.gypi
File content/content_common.gypi (left):

https://codereview.chromium.org/15658004/diff/13001/content/content_common.gy...
content/content_common.gypi:173: 'common/fileapi/webblobregistry_impl.h',
i've got the webblobregistry moving in the works in the mvblob cl, so we can
pull it from this cl

https://codereview.chromium.org/15658004/diff/13001/webkit/support/webkit_sup...
File webkit/support/webkit_support.gypi (right):

https://codereview.chromium.org/15658004/diff/13001/webkit/support/webkit_sup...
webkit/support/webkit_support.gypi:121:
'<(DEPTH)/webkit/renderer/webkit_renderer_storage.gypi:webkit_renderer_storage',
bah... gyp voodoo... should this and compositor_bindings be bundled together &|
handled similarly?

[jamesr, 2013-05-24 20:46:25]

https://codereview.chromium.org/15658004/diff/13001/webkit/renderer/webkit_re...
File webkit/renderer/webkit_renderer_storage.gypi (right):

https://codereview.chromium.org/15658004/diff/13001/webkit/renderer/webkit_re...
webkit/renderer/webkit_renderer_storage.gypi:1: # Copyright 2013 The Chromium
Authors. All rights reserved.
can you make this a real .gyp instead of a gypi?

https://codereview.chromium.org/15658004/diff/13001/webkit/support/webkit_sup...
File webkit/support/webkit_support.gypi (right):

https://codereview.chromium.org/15658004/diff/13001/webkit/support/webkit_sup...
webkit/support/webkit_support.gypi:121:
'<(DEPTH)/webkit/renderer/webkit_renderer_storage.gypi:webkit_renderer_storage',
On 2013/05/24 20:41:59, michaeln wrote:
> bah... gyp voodoo... should this and compositor_bindings be bundled together
&|
> handled similarly?

I don't think it's legal gyp to specify a target inside a gypi, anyway.  you
have to pick a gyp that lists the target.

personally gypis are more trouble than they are worth unless they're used just
for common, simple variables (like filelists) - having targets inside gypis is
very confusing.

[kinuko, 2013-05-27 08:08:48]

Sorry, the change wasn't ready for review... (now it is).

I've realized that I can't make webkit_renderer_storage target yet until we make
its own target for webkit_common_storage, since renderer code depends on common
module. So in this CL I deferred adding webkit_renderer_storage.gyp(i).  PTAL
thanks,

https://codereview.chromium.org/15658004/diff/13001/content/content_common.gypi
File content/content_common.gypi (left):

https://codereview.chromium.org/15658004/diff/13001/content/content_common.gy...
content/content_common.gypi:173: 'common/fileapi/webblobregistry_impl.h',
On 2013/05/24 20:41:59, michaeln wrote:
> i've got the webblobregistry moving in the works in the mvblob cl, so we can
> pull it from this cl

Done.

https://codereview.chromium.org/15658004/diff/13001/webkit/renderer/webkit_re...
File webkit/renderer/webkit_renderer_storage.gypi (right):

https://codereview.chromium.org/15658004/diff/13001/webkit/renderer/webkit_re...
webkit/renderer/webkit_renderer_storage.gypi:1: # Copyright 2013 The Chromium
Authors. All rights reserved.
On 2013/05/24 20:46:25, jamesr wrote:
> can you make this a real .gyp instead of a gypi?

Will do when I add this (I dropped this file from this CL)

[jamesr, 2013-05-27 23:33:21]

lgtm

[kinuko, 2013-05-28 02:10:13]

TBR-ing thestig@ and avi@ for chrome/ and content/ changes. Thx,

[kinuko, 2013-05-28 04:59:34]

Committed patchset #6 manually as r202501.