[Webkit-unassigned] [Bug 97839] New: [EFL][WK2] Make relation between WebContext and {VibrationProvider, BatteryProvider, NetworkInfoProvider}.

[bugzilla-daemon at webkit.org]

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

           Summary: [EFL][WK2] Make relation between WebContext and
                    {VibrationProvider, BatteryProvider,
                    NetworkInfoProvider}.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit EFL
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: bw80.lee at samsung.com
                CC: demarchi at webkit.org


There is an unclear relation between WebContext and {VibrationProvider, BatteryProvider, NetworkInfoProvider}

VibrationProvider has WKRetainPtr<WKVibrationRef> m_wkVibrationRef which has reference for object WebVibrationProxy.

But the WebVibrationProxy instance is a member of WebContext which also has RefPtr<WebVibrationProxy>

Two classes are sharing the reference for WebVibrationProxy but there is no relation between the two.

This can make a problem that,
When WebContext if deleted befor the VibrationProvider, WebVibrationProxy will be alive without WebContext,
because VibrationProvider also has the reference for WebVibrationProxy.

This is a problem because WebVibrationProxy should be a member of WebContext.

To prevent this,
there should be a relation between VibrationProvider and WebContext.

(And same about BatteryProvider, NetworkInfoProvider)

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