Explicitly reject storage access if requesting security origin is unique

Explicitly reject storage access if requesting security origin is unique

This is a follow-up fix for https://codereview.chromium.org/46583005/
(Send Allow{Database,FileSystem,IndexedDB} sync IPCs
directly from worker threads)

I needed to default behavior to allow storage access when NULL proxy
is returned to make content_shell work, and therefore needed not to
return NULL when origin.isUnique()==true case.

BUG=none
R=jam,michaeln
TBR=jam

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

[kinuko, 2013-11-01 09:37:58]

Sorry, I needed this small fix to make the blink code default to 'allow' if no
proxy is given (to make layout tests work). Can you take a look?

[michaeln, 2013-11-01 18:22:45]

lgtm

https://codereview.chromium.org/55433006/diff/1/content/worker/websharedworke...
File content/worker/websharedworkerclient_proxy.cc (left):

https://codereview.chromium.org/55433006/diff/1/content/worker/websharedworke...
content/worker/websharedworkerclient_proxy.cc:85: return NULL;
right... no proxy means "allowed" by default which is not what we want for
unique origins

[commit-bot: I haz the power, 2013-11-04 03:39:58]

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

[commit-bot: I haz the power, 2013-11-04 03:58:39]

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

[kinuko, 2013-11-04 04:29:09]

TBR-ing jam for content/ owner stamp.

[commit-bot: I haz the power, 2013-11-04 04:29:27]

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

[commit-bot: I haz the power, 2013-11-04 07:04:26]

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

[commit-bot: I haz the power, 2013-11-04 13:56:19]

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

[commit-bot: I haz the power, 2013-11-04 16:18:31]

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

[commit-bot: I haz the power, 2013-11-05 01:14:18]

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

[commit-bot: I haz the power, 2013-11-05 03:17:14]

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

[commit-bot: I haz the power, 2013-11-05 06:00:32]

Change committed as 232922