Deprecate the webkit-prefixed standard IndexedDB entry points

Source/core/frame/UseCounter.cpp

 
 /*
  * Copyright (C) 2012 Google, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 679 matching lines @@
 
     case PrefixedVideoExitFullScreen:
         return "'HTMLVideoElement.webkitExitFullScreen()' is deprecated. Please use 'Document.exitFullscreen()' and 'Document.webkitExitFullscreen()' instead.";
 
     case MediaErrorEncrypted:
         return "'MediaError.MEDIA_ERR_ENCRYPTED' is deprecated. This error code is never used.";
 
     case PrefixedGamepad:
         return "'navigator.webkitGetGamepads' is deprecated. Please use 'navigator.getGamepads' instead.";
 
+    case PrefixedIndexedDB:
+        return "'webkitIndexedDB' is deprecated. Please use the standard 'indexedDB' instead.";

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

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]

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.

+
+    case PrefixedIDBCursorConstructor:
+        return "'webkitIDBCursor' is deprecated. Please use the standard 'IDBCursor' instead.";
+
+    case PrefixedIDBDatabaseConstructor:
+        return "'webkitIDBDatabase' is deprecated. Please use the standard 'IDBDatabase' instead.";
+
+    case PrefixedIDBFactoryConstructor:
+        return "'webkitIDBFactory' is deprecated. Please use the standard 'IDBFactory' instead.";
+
+    case PrefixedIDBIndexConstructor:
+        return "'webkitIDBIndex' is deprecated. Please use the standard 'IDBIndex' instead.";
+
+    case PrefixedIDBKeyRangeConstructor:
+        return "'webkitIDBKeyRange' is deprecated. Please use the standard 'IDBKeyRange' instead.";
+
+    case PrefixedIDBObjectStoreConstructor:
+        return "'webkitIDBObjectStore' is deprecated. Please use the standard 'IDBObjectStore' instead.";
+
+    case PrefixedIDBRequestConstructor:
+        return "'webkitIDBRequest' is deprecated. Please use the standard 'IDBRequest' instead.";
+
+    case PrefixedIDBTransactionConstructor:
+        return "'webkitIDBTransaction' is deprecated. Please use the standard 'IDBTransaction' instead.";
+
     case PrefixedRequestAnimationFrame:
         return "'webkitRequestAnimationFrame' is vendor-specific. Please use the standard 'requestAnimationFrame' instead.";
 
     case PrefixedCancelAnimationFrame:
         return "'webkitCancelAnimationFrame' is vendor-specific. Please use the standard 'cancelAnimationFrame' instead.";
 
     case PrefixedCancelRequestAnimationFrame:
         return "'webkitCancelRequestAnimationFrame' is vendor-specific. Please use the standard 'cancelAnimationFrame' instead.";
 
     case DocumentCreateAttributeNS:
@@ skipping 80 matching lines @@
 UseCounter* UseCounter::getFrom(const StyleSheetContents* sheetContents)
 {
     // FIXME: We may want to handle stylesheets that have multiple owners
     //        http://crbug.com/242125
     if (sheetContents && sheetContents->hasSingleOwnerNode())
         return getFrom(sheetContents->singleOwnerDocument());
     return 0;
 }
 
 } // namespace blink