Move IndexedDBContext into the StoragePartition and ensure isolation.

Move IndexedDBContext into the StoragePartition and ensure isolation.

This pickups up from http://codereview.chromium.org/10836216.  After this, IndexedDB should be properly isolated in ChromeApps and Isolated Sites.

TBR=kalman,sky,mkwst 
BUG=85121
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157045

[awong, 2012-09-15 00:45:42]

Another one!

[Charlie Reis, 2012-09-15 00:55:58]

Nice and simple.  LGTM, with indent fixes.

https://codereview.chromium.org/10919307/diff/9002/chrome/browser/extensions/...
File chrome/browser/extensions/data_deleter.cc (right):

https://codereview.chromium.org/10919307/diff/9002/chrome/browser/extensions/...
chrome/browser/extensions/data_deleter.cc:91:
BrowserContext::GetDefaultStoragePartition(profile);
nit: wrong indent

https://codereview.chromium.org/10919307/diff/9002/chrome/browser/ui/webui/op...
File chrome/browser/ui/webui/options/cookies_view_handler.cc (right):

https://codereview.chromium.org/10919307/diff/9002/chrome/browser/ui/webui/op...
chrome/browser/ui/webui/options/cookies_view_handler.cc:197:
storage_partition->GetIndexedDBContext();
nit: wrong indent

https://codereview.chromium.org/10919307/diff/9002/content/browser/browser_co...
File content/browser/browser_context.cc (right):

https://codereview.chromium.org/10919307/diff/9002/content/browser/browser_co...
content/browser/browser_context.cc:177:
BrowserContext::GetDefaultStoragePartition(browser_context);
nit: wrong indent

https://codereview.chromium.org/10919307/diff/9002/content/browser/in_process...
File content/browser/in_process_webkit/indexed_db_browsertest.cc (right):

https://codereview.chromium.org/10919307/diff/9002/content/browser/in_process...
content/browser/in_process_webkit/indexed_db_browsertest.cc:199:
GetIndexedDBContext();
nit: needs another 4 spaces

https://codereview.chromium.org/10919307/diff/9002/content/browser/renderer_h...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/10919307/diff/9002/content/browser/renderer_h...
content/browser/renderer_host/render_process_host_impl.cc:552:
storage_partition_impl_->GetIndexedDBContext()));
nit: wrong indent

[awong, 2012-09-15 01:19:37]

TBRing owners changes for kalman, sky, and mkwst because this is just a viral
propagation of a changed API.

FYI, you guys may see a couple more of these as we move FileSystemContext, and
AppCache through.

http://codereview.chromium.org/10919307/diff/9002/chrome/browser/extensions/d...
File chrome/browser/extensions/data_deleter.cc (right):

http://codereview.chromium.org/10919307/diff/9002/chrome/browser/extensions/d...
chrome/browser/extensions/data_deleter.cc:91:
BrowserContext::GetDefaultStoragePartition(profile);
On 2012/09/15 00:55:58, creis wrote:
> nit: wrong indent

Done.

http://codereview.chromium.org/10919307/diff/9002/chrome/browser/ui/webui/opt...
File chrome/browser/ui/webui/options/cookies_view_handler.cc (right):

http://codereview.chromium.org/10919307/diff/9002/chrome/browser/ui/webui/opt...
chrome/browser/ui/webui/options/cookies_view_handler.cc:197:
storage_partition->GetIndexedDBContext();
On 2012/09/15 00:55:58, creis wrote:
> nit: wrong indent

Done.

http://codereview.chromium.org/10919307/diff/9002/content/browser/browser_con...
File content/browser/browser_context.cc (right):

http://codereview.chromium.org/10919307/diff/9002/content/browser/browser_con...
content/browser/browser_context.cc:177:
BrowserContext::GetDefaultStoragePartition(browser_context);
On 2012/09/15 00:55:58, creis wrote:
> nit: wrong indent

Done.

http://codereview.chromium.org/10919307/diff/9002/content/browser/in_process_...
File content/browser/in_process_webkit/indexed_db_browsertest.cc (right):

http://codereview.chromium.org/10919307/diff/9002/content/browser/in_process_...
content/browser/in_process_webkit/indexed_db_browsertest.cc:199:
GetIndexedDBContext();
On 2012/09/15 00:55:58, creis wrote:
> nit: needs another 4 spaces

Done.

http://codereview.chromium.org/10919307/diff/9002/content/browser/renderer_ho...
File content/browser/renderer_host/render_process_host_impl.cc (right):

http://codereview.chromium.org/10919307/diff/9002/content/browser/renderer_ho...
content/browser/renderer_host/render_process_host_impl.cc:552:
storage_partition_impl_->GetIndexedDBContext()));
On 2012/09/15 00:55:58, creis wrote:
> nit: wrong indent

Done.

[commit-bot: I haz the power, 2012-09-15 01:20:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10919307/2003

[commit-bot: I haz the power, 2012-09-15 02:02:58]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-09-15 02:38:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10919307/6004

[commit-bot: I haz the power, 2012-09-15 04:30:41]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-09-15 05:18:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10919307/6004

[commit-bot: I haz the power, 2012-09-15 05:19:03]

Failed to apply patch for content/browser/browser_context.cc:
While running patch -p1 --forward --force;
patching file content/browser/browser_context.cc
Hunk #1 FAILED at 103.
Hunk #2 succeeded at 173 (offset -7 lines).
Hunk #3 succeeded at 182 (offset -7 lines).
1 out of 3 hunks FAILED -- saving rejects to file
content/browser/browser_context.cc.rej

[Use mkwst_at_chromium.org plz., 2012-09-15 07:43:19]

browsing_data LGTM.

[Mike West, 2012-09-15 07:44:13]

On 2012/09/15 07:43:19, Use mkwst_at_chromium.org. wrote:
> browsing_data LGTM.

Er. LGTM from the right account. :)

[commit-bot: I haz the power, 2012-09-16 05:26:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10919307/9030

[commit-bot: I haz the power, 2012-09-16 07:31:44]

Change committed as 157045