RESOLVED FIXED234184
ContextDestructionObserver::m_scriptExecutionContext should be private 
https://bugs.webkit.org/show_bug.cgi?id=234184
Summary ContextDestructionObserver::m_scriptExecutionContext should be private
Chris Dumez
Reported 2021-12-10 15:56:05 PST
ContextDestructionObserver::m_scriptExecutionContext should be private. It is poor encapsulation to have protected data members.
Attachments
Patch (31.89 KB, patch)
2021-12-10 15:58 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (31.89 KB, patch)
2021-12-11 15:26 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2021-12-10 15:58:02 PST
Created attachment 446836 [details] Patch
youenn fablet
Comment 2 2021-12-11 09:16:03 PST
Comment on attachment 446836 [details] Patch Some pre-existing places are using *scriptExecutionContext() and it is not clear whether it is guaranteed to not be null. View in context: https://bugs.webkit.org/attachment.cgi?id=446836&action=review > Source/WebCore/Modules/encryptedmedia/MediaKeySystemRequest.cpp:65 > + auto context = scriptExecutionContext(); auto*
Chris Dumez
Comment 3 2021-12-11 15:26:04 PST
Created attachment 446908 [details] Patch
EWS
Comment 4 2021-12-11 16:30:17 PST
Committed r286918 (245144@main): <https://commits.webkit.org/245144@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 446908 [details].
Radar WebKit Bug Importer
Comment 5 2021-12-11 16:31:18 PST
<rdar://problem/86370839>
Note You need to log in before you can comment on or make changes to this bug.