Use chromium logic for database identifier<->origin conversions

Use chromium logic for database identifier<->origin conversions

This switches over to using the logic added in r206815 for database id /
origin identification instead of WebSecurityOrigin. I'll delete the old code
after this has landed.

BUG=243095
TBR=thakis

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

[jamesr, 2013-06-18 21:37:49]

Michael - please review.  I've run content_unittests / content_browsertests with
a version of this patch that called the old and the new functions and DCHECK()d
that they get the same answer.

[jamesr, 2013-06-18 21:38:14]

Fixing title

[jsbell, 2013-06-18 21:49:50]

https://codereview.chromium.org/16879013/diff/3001/content/browser/indexed_db...
File content/browser/indexed_db/indexed_db_context_impl.cc (right):

https://codereview.chromium.org/16879013/diff/3001/content/browser/indexed_db...
content/browser/indexed_db/indexed_db_context_impl.cc:37:
FILE_PATH_LITERAL(".indexeddb");
FYI, these string constants are duplicated in indexed_db_backing_store.cc - they
were duplicated in Blink and we haven't cleaned up to de-dupe them.

Without digging into that tangled mess just now, the right thing to do is send
them over from the Context anyway, so this change is great.

[michaeln, 2013-06-18 23:20:46]

lgtm

[commit-bot: I haz the power, 2013-06-18 23:29:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/16879013/14001

[jamesr, 2013-06-18 23:39:53]

TBR'ing thakis for chrome/browser/extensions/extension_service_unittest.cc and
content/browser/renderer_host/database_message_filter.cc since this is an API
change with owners approval

[commit-bot: I haz the power, 2013-06-18 23:40:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/16879013/14001

[commit-bot: I haz the power, 2013-06-19 06:19:37]

Change committed as 207191

[Nico, 2013-06-19 18:28:24]

On 2013/06/18 23:39:53, jamesr wrote:
> TBR'ing thakis for chrome/browser/extensions/extension_service_unittest.cc and
> content/browser/renderer_host/database_message_filter.cc since this is an API
> change with owners approval

lgtm

That's cool, but somehow I didn't receive this message. The first message I got
from this thread was the "CQ is trying da patch" message, which isn't very
informative.