86127 2012-05-10 12:29:13 -0700 IndexedDB: IDBDatabase properties not snapshotted on connection close 2012-06-22 15:23:32 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 88467 1 jsbell webkit-unassigned oldest_to_newest 620856 0 jsbell 2012-05-10 12:29:13 -0700 The spec sayeth: http://dvcs.w3.org/hg/IndexedDB/raw-file/tip/Overview.html#idl-def-IDBDatabase IDBDatabase.objectStoreNames: "Once the closePending flag is set on the connection, this function must return a snapshot of the list of names of the object stores taken at the time when the close method was called." ... "Even if another connection is later used to modify the version, that attribute on closed instances are not changed." IDBDatabase.version; "On getting, this attribute must return the version of the database when this IDBDatabase instance was created. When a IDBDatabase instance is created, this is always the number passed as the version argument passed to the open call used to create the IDBDatabase instance. This value remains constant for the lifetime of the IDBDatabase object. If the connection is closed, this attribute represents a snapshot of the version that the database had when the connection was closed. Even if another connection is later used to modify the version, that attribute on closed instances are not changed." WebKit currently always queries the back end, even for closed connections. For example, running the attached script yields: open - expecting 1: 1 open - expecting 1: 1 open - expecting 2: 2 open - expecting 2: 2 closed - expecting 1: 2 closed - expecting 1: 2 closed - expecting 2: 2 closed - expecting 2: 2 Note the "closed - expecting 1: 2" lines that show the value was not snapshotted 620859 1 141225 jsbell 2012-05-10 12:29:35 -0700 Created attachment 141225 Repro case 632161 2 jsbell 2012-05-23 16:18:44 -0700 Implementing http://crbug.com/129470 would make this relatively easy. 655831 3 jsbell 2012-06-22 15:23:32 -0700 Fixed by http://trac.webkit.org/changeset/121059 141225 2012-05-10 12:29:35 -0700 2012-05-10 12:29:35 -0700 Repro case nosnap.html text/html 1394 jsbell PCFET0NUWVBFIGh0bWw+CjxzY3JpcHQ+CnZhciBkYmgxLCBkYmgyOwoKd2Via2l0SW5kZXhlZERC LmRlbGV0ZURhdGFiYXNlKCJub3NuYXAiKS5vbnN1Y2Nlc3MgPSBmdW5jdGlvbiAoZSkgewogIGRv Rmlyc3RPcGVuKCk7Cn07CgpmdW5jdGlvbiBkb0ZpcnN0T3BlbigpIHsKICB3ZWJraXRJbmRleGVk REIub3Blbigibm9zbmFwIikub25zdWNjZXNzID0gZnVuY3Rpb24gKGUpIHsKICAgIGRiaDEgPSBl LnRhcmdldC5yZXN1bHQ7CiAgICBkYmgxLnNldFZlcnNpb24oIjEiKS5vbnN1Y2Nlc3MgPSBmdW5j dGlvbiAoZSkgewogICAgICBkYmgxLmNyZWF0ZU9iamVjdFN0b3JlKCJzdG9yZTEiKTsKICAgICAg ZG9jdW1lbnQud3JpdGUoIjxsaT5vcGVuIC0gZXhwZWN0aW5nIDE6ICIgKyBkYmgxLm9iamVjdFN0 b3JlTmFtZXMubGVuZ3RoKTsKICAgICAgZG9jdW1lbnQud3JpdGUoIjxsaT5vcGVuIC0gZXhwZWN0 aW5nIDE6ICIgKyBkYmgxLnZlcnNpb24pOwogICAgICBlLnRhcmdldC5yZXN1bHQub25jb21wbGV0 ZSA9IGZ1bmN0aW9uICgpIHsKICAgICAgICBkYmgxLmNsb3NlKCk7CiAgICAgICAgZG9TZWNvbmRP cGVuKCk7CiAgICAgIH07CiAgICB9OyAgCiAgfTsKfQoKZnVuY3Rpb24gZG9TZWNvbmRPcGVuKCkg ewogIHdlYmtpdEluZGV4ZWREQi5vcGVuKCJub3NuYXAiKS5vbnN1Y2Nlc3MgPSBmdW5jdGlvbiAo ZSkgewogICAgZGJoMiA9IGUudGFyZ2V0LnJlc3VsdDsKICAgIGRiaDIuc2V0VmVyc2lvbigiMiIp Lm9uc3VjY2VzcyA9IGZ1bmN0aW9uIChlKSB7CiAgICAgIGRiaDIuY3JlYXRlT2JqZWN0U3RvcmUo InN0b3JlMiIpOwogICAgICBkb2N1bWVudC53cml0ZSgiPGxpPm9wZW4gLSBleHBlY3RpbmcgMjog IiArIGRiaDIub2JqZWN0U3RvcmVOYW1lcy5sZW5ndGgpOwogICAgICBkb2N1bWVudC53cml0ZSgi PGxpPm9wZW4gLSBleHBlY3RpbmcgMjogIiArIGRiaDIudmVyc2lvbik7CiAgICAgIGUudGFyZ2V0 LnJlc3VsdC5vbmNvbXBsZXRlID0gZnVuY3Rpb24gKCkgewogICAgICAgIGRiaDIuY2xvc2UoKTsK ICAgICAgICBjaGVja1N0YXRlKCk7CiAgICAgIH07CiAgICB9OyAgCiAgfTsKfQoKZnVuY3Rpb24g Y2hlY2tTdGF0ZSgpIHsKICBkb2N1bWVudC53cml0ZSgiPGxpPmNsb3NlZCAtIGV4cGVjdGluZyAx OiAiICsgZGJoMS5vYmplY3RTdG9yZU5hbWVzLmxlbmd0aCk7CiAgZG9jdW1lbnQud3JpdGUoIjxs aT5jbG9zZWQgLSBleHBlY3RpbmcgMTogIiArIGRiaDEudmVyc2lvbik7CiAgZG9jdW1lbnQud3Jp dGUoIjxsaT5jbG9zZWQgLSBleHBlY3RpbmcgMjogIiArIGRiaDIub2JqZWN0U3RvcmVOYW1lcy5s ZW5ndGgpOwogIGRvY3VtZW50LndyaXRlKCI8bGk+Y2xvc2VkIC0gZXhwZWN0aW5nIDI6ICIgKyBk YmgyLnZlcnNpb24pOwp9Cjwvc2NyaXB0Pgo=