Deprecate the webkit-prefixed standard IndexedDB entry points

Deprecate the webkit-prefixed standard IndexedDB entry points

We've shipped unprefixed (e.g. indexedDB, IDBKeyRange, etc) for almost
two years. Time to at least try to wean the web off of them.

Intent to Deprecate:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/6TD2jjqAJs0

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179058

[jsbell, 2014-07-23 20:06:10]

Still waiting on the requisite approval on the "Intent", so not landing this
yet.

I could split this into two CLs if desired:

(1) Remove Blink dependencies on webkitIndexedDB and friends (places that used
the prefixed versions but don't need to)
(2) Actually do the deprecation

[jsbell, 2014-07-23 20:12:52]

https://codereview.chromium.org/415753002/diff/1/LayoutTests/storage/indexedd...
File LayoutTests/storage/indexeddb/resources/shared.js (left):

https://codereview.chromium.org/415753002/diff/1/LayoutTests/storage/indexedd...
LayoutTests/storage/indexeddb/resources/shared.js:27: function
removeVendorPrefixes()
Since this is used by nearly all of the storage/indexeddb tests and logs, I was
planning to remove this in a separate CL.

[eseidel, 2014-07-23 20:18:31]

lgtm

https://codereview.chromium.org/415753002/diff/1/LayoutTests/http/tests/inspe...
File LayoutTests/http/tests/inspector/indexeddb/indexeddb-test.js (left):

https://codereview.chromium.org/415753002/diff/1/LayoutTests/http/tests/inspe...
LayoutTests/http/tests/inspector/indexeddb/indexeddb-test.js:103: var indexedDB
= window.indexeddb || window.webkitIndexedDB;
I might have left these two as blank lines to avoid changing all the tests. :p

https://codereview.chromium.org/415753002/diff/1/Source/core/frame/UseCounter...
File Source/core/frame/UseCounter.cpp (right):

https://codereview.chromium.org/415753002/diff/1/Source/core/frame/UseCounter...
Source/core/frame/UseCounter.cpp:701: return "'webkitIndexedDB' is deprecated.
Please use the standard 'indexedDB' instead.";
I'm suprised we don't have a helper fucntion for generating these strings.  e.g.
replacedBy('webkitIndexDB', 'indexDB')

[jsbell, 2014-07-23 21:12:42]

https://codereview.chromium.org/415753002/diff/1/Source/core/frame/UseCounter...
File Source/core/frame/UseCounter.cpp (right):

https://codereview.chromium.org/415753002/diff/1/Source/core/frame/UseCounter...
Source/core/frame/UseCounter.cpp:701: return "'webkitIndexedDB' is deprecated.
Please use the standard 'indexedDB' instead.";
On 2014/07/23 20:18:31, eseidel wrote:
> I'm suprised we don't have a helper fucntion for generating these strings. 
e.g.
> replacedBy('webkitIndexDB', 'indexDB')

Good idea. I went with the generic "'%s' is deprecated. Please use '%s'
instead." template and updated the handful of other matching strings.

I left other variants (e.g. "...is vendor-specific.", "Please use the
standard...") alone for now but added a FIXME. Even if we're not gaining much
with that additional wording, I didn't want to rebase non-IDB test in this CL.

[cmumford, 2014-07-23 23:51:59]

lgtm. I'm guessing that these four files will be modified once these are fully
removed:

LayoutTests/virtual/stable/webexposed/global-constructors-listing-dedicated-worker-expected.txt
LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt
LayoutTests/virtual/stable/webexposed/global-constructors-listing-shared-worker-expected.txt
LayoutTests/webexposed/global-constructors-listing-shared-worker-expected.txt

[jsbell, 2014-07-23 23:55:02]

On 2014/07/23 23:51:59, cmumford wrote:
> lgtm. I'm guessing that these four files will be modified once these are fully
> removed:
> 
>
LayoutTests/virtual/stable/webexposed/global-constructors-listing-dedicated-worker-expected.txt
> LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt
>
LayoutTests/virtual/stable/webexposed/global-constructors-listing-shared-worker-expected.txt
> LayoutTests/webexposed/global-constructors-listing-shared-worker-expected.txt

Correct. Also, the storage/indexeddb/unprefix.html and
storage/indexeddb/unprefix-workers.html references to webkitIndexedDB can be
moved to removed.html.

[jsbell, 2014-07-28 21:22:35]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2014-07-28 21:23:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jsbell@chromium.org/415753002/40001

[commit-bot: I haz the power, 2014-07-28 21:28:06]

Change committed as 179058