[Webkit-unassigned] [Bug 248173] New: Workers: Simplify early dedicated worker termination

[bugzilla-daemon at webkit.org]

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

            Bug ID: 248173
           Summary: Workers: Simplify early dedicated worker termination
           Product: WebKit
           Version: Safari Technology Preview
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: New Bugs
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com

Hi Team,

While going through Blink commits, I found another simplification in workers termination, which can be brought to Webkit:

Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=180532

Although there is no test case to confirm whether Webkit suffer from some issue due to it or not but since it is just simplification, it seems to be safe. I can do draft PR and run via EWS to see if there is any issue across multiple platforms.

https://github.com/WebKit/WebKit/blob/a845b7e4ec0490d1d8f5d7e7f5ddf44fef0b6f17/Source/WebCore/workers/WorkerMessagingProxy.cpp#L330

^ Remove this bit from here and move it here:

https://github.com/WebKit/WebKit/blob/a845b7e4ec0490d1d8f5d7e7f5ddf44fef0b6f17/Source/WebCore/workers/WorkerMessagingProxy.cpp#LL122

Just wanted to raise bug.

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20221121/e9d2bcc8/attachment-0001.htm>