Bug 79508 – Switch indexeddb to use supplemental IDL for WorkerContext

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: New Bugs

[Mark Pilgrim (Google)]

Switch indexeddb to use supplemental IDL for WorkerContext

[Mark Pilgrim (Google)]

Created attachment 128758 [details]
[WIP] Patch

[Mark Pilgrim (Google)]

Created attachment 128759 [details]
[WIP] Patch

[Adam Barth]

Comment on attachment 128758 [details]
[WIP] Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=128758&action=review

> Source/WebCore/workers/WorkerContext.h:177
>  #if ENABLE(INDEXED_DATABASE)
> -        IDBFactory* webkitIndexedDB() const;
> +        IDBFactory* idbFactory() { return m_idbFactory.get(); }
> +        void setIDBFactory(PassRefPtr<IDBFactory>);
> +        IDBFactoryBackendInterface* idbFactoryBackendInterface() { return m_idbFactoryBackendInterface.get(); }
> +        void setIDBFactoryBackendInterface(PassRefPtr<IDBFactoryBackendInterface>);
>  #endif

We should be able to remove the ENABLE(INDEXED_DATABASE) from WorkerContext using something analogous to PageSupplement.  It might be slightly trickier though because WorkerContext is used on many threads.

[WebKit Review Bot]

Comment on attachment 128759 [details]
[WIP] Patch

Attachment 128759 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11627106

[Mark Pilgrim (Google)]

Created attachment 128766 [details]
Patch

[Hajime Morrita]

Comment on attachment 128766 [details]
Patch

Looks like abarth snached  this at Bug 80094 :-/

[Mark Pilgrim (Google)]

*** This bug has been marked as a duplicate of bug 80094 ***