[Webkit-unassigned] [Bug 40112] New: Database callbacks are made using the ScriptExecutionContext of the frame that owns the Database object, rather than that of the caller

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 3 04:38:17 PDT 2010


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

           Summary: Database callbacks are made using the
                    ScriptExecutionContext of the frame that owns the
                    Database object, rather than that of the caller
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: steveblock at google.com
                CC: andreip at google.com, michaeln at google.com,
                    dglazkov at chromium.org, jorlow at chromium.org,
                    dumi at chromium.org, ericu at chromium.org


Database callbacks are made using the ScriptExecutionContext of the frame that owns the Database object. Instead, callbacks should be made using the ScriptContext of the caller that supplied the callback function.

Also, it looks like the Database object holds a RefPtr is to it's owning frame's context, but I don't think that the Database object should prolong the lifetime of the ScriptExecution context. See https://bugs.webkit.org/show_bug.cgi?id=39388#c6 Instead, if the relevant context no longer exists, the callbacks should simply not be made.

I tried writing a LayoutTest to demonstrate this, but had difficulty as the Database object already prevents callbacks once the frame owning the ScriptExecutionContext has gone away, via its stop() method.

See Bug 39879 for related discussion

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



More information about the webkit-unassigned mailing list