[Webkit-unassigned] [Bug 85009] New: ScriptStateProtectedPtr should not keep a strong reference to the context

[bugzilla-daemon at webkit.org]

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

           Summary: ScriptStateProtectedPtr should not keep a strong
                    reference to the context
           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: jochen at chromium.org
                CC: antonm at chromium.org


ScriptStateProtectedPtr keeps a strong ref to the javascript context. It's only used from the Inspector to store script arguments of console messages. While only a certain amount of console message (1000) are kept, this can keep up to 1000 different contexts alive which might easily exceed the available memory.

I think instead, the ScriptStateProtectedPtr should keep a weak reference, and the inspector should cope with the arguments being gone when the context died.

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