IndexedDB: IDL types defined in spec should be visible to scripts
Created attachment 168266 [details] Patch
alecflett@, dgrogan@ - please take a look. With this we pass http://w3c-test.org/webapps/IndexedDB/tests/submissions/Opera/idbtransaction.htm
LGTM I don't know why this is necessary, but I believe you that it is. Seems harmless. removeVendorPrefixes() in shared.js has a similar check (shouldBeTrueQuiet...). You might want to update that instead of or in addition to the layout test you have here.
> I don't know why this is necessary, but I believe you that it is. Seems harmless. You don't know why exposing these properties is necessary from an API point of view or why we need to declare them explicitly in the IDL from an implementation point of view?
(In reply to comment #4) > > I don't know why this is necessary, but I believe you that it is. Seems harmless. > > You don't know why exposing these properties is necessary from an API point of view or why we need to declare them explicitly in the IDL from an implementation point of view? The first.
Typically its so web pages can manipulate the prototypes of these objects, for example to shim in future versions of the API.
(In reply to comment #6) > Typically its so web pages can manipulate the prototypes of these objects, for example to shim in future versions of the API. Got it, that makes sense. Josh also mentioned that this could allow for easier feature detection.
Created attachment 168268 [details] Patch for landing
Comment on attachment 168268 [details] Patch for landing Clearing flags on attachment: 168268 Committed r131096: <http://trac.webkit.org/changeset/131096>
All reviewed patches have been landed. Closing bug.