Move webkit/quota files to webkit/browser/quota or webkit/common/quota

Move webkit/quota files to webkit/browser/quota or webkit/common/quota

This CL includes:
 - Move webkit/quota/quota_{types.{h,cc},callbacks.h} webkit/common/quota/,
 - Move other webkit/quota/ files to webkit/browser/quota/,
 - #include and include guard fix for the move,
 - Split webkit/quota/webkit_quota.gypi to webkit/{browser,common}/quota,
 - Change DEPS to allow #include webkit/common from chrome/browser.

BUG=244363
TEST=should build successfully and should pass deps check.
NOTRY=True

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

[tzik, 2013-05-28 15:00:59]

I'm splitting webkit/quota to webkit/{browser,common}/quota.

Lei: Could you take a look to chrome/ and its DEPS change?
Avi: PTAL to content/?
Kinuko: PTL to webkit/?
Mikhail: PTL to android_webview/?

[kinuko, 2013-05-28 15:19:28]

lgtm

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/DEPS
File chrome/browser/DEPS (right):

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/DEPS#newcode58
chrome/browser/DEPS:58: "+webkit/common/quota",
It might be ok just to add +webkit/common (at line 66) and remove all these

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/browsing_da...
File chrome/browser/browsing_data/browsing_data_quota_helper_impl.h (right):

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/browsing_da...
chrome/browser/browsing_data/browsing_data_quota_helper_impl.h:21: class GURL;
not needed

[Avi (use Gerrit), 2013-05-28 15:22:33]

Just shuffling stuff around? lgtm

[tzik, 2013-05-28 15:35:31]

Kinuko, Avi: Thanks!

This CL also touches *_messages.h.

Cris: Could you PTAL to *_messages.h below? It's just .h moving for them.
content/common/fileapi/file_system_messages.h
content/common/quota_messages.h

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/DEPS
File chrome/browser/DEPS (right):

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/DEPS#newcode58
chrome/browser/DEPS:58: "+webkit/common/quota",
On 2013/05/28 15:19:28, kinuko wrote:
> It might be ok just to add +webkit/common (at line 66) and remove all these

Done.

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/browsing_da...
File chrome/browser/browsing_data/browsing_data_quota_helper_impl.h (right):

https://codereview.chromium.org/16010007/diff/8018/chrome/browser/browsing_da...
chrome/browser/browsing_data/browsing_data_quota_helper_impl.h:21: class GURL;
On 2013/05/28 15:19:28, kinuko wrote:
> not needed

Done.
Thanks! This is a rebase failure.

[mnaganov (inactive), 2013-05-28 16:14:10]

android_webview LGTM

[Lei Zhang, 2013-05-28 18:04:24]

chrome/ lgtm

[Cris Neckar, 2013-05-29 20:29:28]

IPC changes rubber stamp LGTM

[commit-bot: I haz the power, 2013-05-29 23:08:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/16010007/18001

[commit-bot: I haz the power, 2013-05-29 23:08:50]

Failed to apply patch for chrome/browser/DEPS:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/DEPS
  Hunk #1 FAILED at 53.
  1 out of 1 hunk FAILED -- saving rejects to file chrome/browser/DEPS.rej

Patch:       chrome/browser/DEPS
Index: chrome/browser/DEPS
diff --git a/chrome/browser/DEPS b/chrome/browser/DEPS
index
35a22808997842adb047a67ba3c1878d6af1d507..93fe00bcca25dea112b338304423f80fbc893525
100644
--- a/chrome/browser/DEPS
+++ b/chrome/browser/DEPS
@@ -53,16 +53,13 @@ include_rules = [
   "+webkit/appcache",
   "+webkit/base/origin_url_conversions.h",
   "+webkit/blob",
-  "+webkit/common/fileapi",
-  "+webkit/common/plugins",  # Common plugins functionality.
-  "+webkit/common/user_agent",
   "+webkit/dom_storage/dom_storage_types.h",
   "+webkit/plugins",  # Defines some types that are marshalled over IPC.
-  "+webkit/quota",
 
   # TODO(kinuko): Remove all other webkit/* dependencies after we cleaned
   # up webkit/. crbug.com/239710
   "+webkit/browser",
+  "+webkit/common",
 
   # webkit/glue files are listed individually since they aren't conceptually
   # grouped like the other webkit/ files, and can therefore be tackled one at

[commit-bot: I haz the power, 2013-05-30 02:07:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/16010007/28001

[commit-bot: I haz the power, 2013-05-30 03:41:08]

Retried try job too often on win_rel for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-05-30 04:08:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tzik@chromium.org/16010007/28001

[commit-bot: I haz the power, 2013-05-30 05:12:49]

Change committed as 203082