[Webkit-unassigned] [Bug 101483] New: IndexedDB: Replace use of ScriptExecutionContext::Task

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=101483

           Summary: IndexedDB: Replace use of ScriptExecutionContext::Task
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jsbell at chromium.org
                CC: dgrogan at chromium.org, alecflett at chromium.org


We use ScriptExecutionContext::Task in IDBTransactionBackendImpl to hold "tasks", but this pulls a lot of machinery in that we don't need e.g. ThreadSafeRefCounted, and so forth.

Consider replacing this usage with something else - either a generic closure class, or operation-specific holders (e.g. ObjectStoreStorageOperation, ObjectStoreRetrievalOperation, etc).

The latter is particularly appealing when we consider moving those to the front end as they would encapsulate the asynchronous logic (e.g. for an "add" you need to do at least one "get" to verify that the value is not present, more "gets" if there are "unique" indexes, then several "puts" - all of which may be async)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.