Migrate webkit/browser/ to storage/browser/

Migrate webkit/browser/ to storage/browser/

using TBR because Darin told me he defers to James' judgement. Using NOTRY because there is one presubmit error (dump_file_system.cc was moved and it spews printf).

BUG=338338
TBR=darin@chromium.org
NOTRY=true

Committed: https://crrev.com/4af8c2120e23d17c1cac3f929c3a1d901c5701ab
Cr-Commit-Position: refs/heads/master@{#293547}

[pilgrim_google, 2014-09-04 14:29:04]

pilgrim@chromium.org changed reviewers:
+ jam@chromium.org, jamesr@chromium.org, michaeln@chromium.org

[jamesr, 2014-09-05 01:02:31]

I didn't mark everything but please go through and sort all of the lists of
things you've updated.

lgtm

https://codereview.chromium.org/539143002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/539143002/diff/1/BUILD.gn#newcode126
BUILD.gn:126: "//storage/browser",
please keep this alphabetical

https://codereview.chromium.org/539143002/diff/1/BUILD.gn#newcode228
BUILD.gn:228: "//storage/browser",
ditto

https://codereview.chromium.org/539143002/diff/1/android_webview/native/webvi...
File android_webview/native/webview_native.gyp (right):

https://codereview.chromium.org/539143002/diff/1/android_webview/native/webvi...
android_webview/native/webview_native.gyp:26:
'../../storage/storage_browser.gyp:storage',
sort please

https://codereview.chromium.org/539143002/diff/1/chrome/browser/BUILD.gn
File chrome/browser/BUILD.gn (right):

https://codereview.chromium.org/539143002/diff/1/chrome/browser/BUILD.gn#newc...
chrome/browser/BUILD.gn:183: "//storage/browser",
sort

https://codereview.chromium.org/539143002/diff/1/chrome/browser/extensions/BU...
File chrome/browser/extensions/BUILD.gn (right):

https://codereview.chromium.org/539143002/diff/1/chrome/browser/extensions/BU...
chrome/browser/extensions/BUILD.gn:75: "//storage/browser",
sort

https://codereview.chromium.org/539143002/diff/1/chrome/browser/ui/BUILD.gn
File chrome/browser/ui/BUILD.gn (right):

https://codereview.chromium.org/539143002/diff/1/chrome/browser/ui/BUILD.gn#n...
chrome/browser/ui/BUILD.gn:111: "//storage/common",
sort both of these, please

https://codereview.chromium.org/539143002/diff/1/chrome/chrome_browser.gypi
File chrome/chrome_browser.gypi (right):

https://codereview.chromium.org/539143002/diff/1/chrome/chrome_browser.gypi#n...
chrome/chrome_browser.gypi:2943: '../storage/storage_browser.gyp:storage',
sort

https://codereview.chromium.org/539143002/diff/1/webkit/browser/storage_brows...
File webkit/browser/storage_browser_export.h (right):

https://codereview.chromium.org/539143002/diff/1/webkit/browser/storage_brows...
webkit/browser/storage_browser_export.h:5: #include
"storage/browser/storage_browser_export.h"
you shouldn't need a forwarding export header since you moved all the headers
that use the export (by definition they're all in the component you moved)

[pilgrim_google, 2014-09-05 01:22:39]

On 2014/09/05 01:02:31, jamesr wrote:
>
https://codereview.chromium.org/539143002/diff/1/webkit/browser/storage_brows...
> webkit/browser/storage_browser_export.h:5: #include
> "storage/browser/storage_browser_export.h"
> you shouldn't need a forwarding export header since you moved all the headers
> that use the export (by definition they're all in the component you moved)

Hahahahahahaha no.

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/med...

[pilgrim_google, 2014-09-05 17:21:24]

The CQ bit was checked by pilgrim@chromium.org

[commit-bot: I haz the power, 2014-09-05 17:22:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pilgrim@chromium.org/539143002/30001

[commit-bot: I haz the power, 2014-09-05 17:30:53]

Committed patchset #4 (id:30001) as 7f18a5464f5cd1b2b9520e23e1e7e93996bca4ae

[commit-bot: I haz the power, 2014-09-10 03:40:27]

Patchset 4 (id:??) landed as
https://crrev.com/4af8c2120e23d17c1cac3f929c3a1d901c5701ab
Cr-Commit-Position: refs/heads/master@{#293547}