Summary: | Fix runtime expose scope check for shadow realms | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Griego <joseph.j.griego> | ||||||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | cdumez, darin, esprehn+autocc, ews-watchlist, kangil.han, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
Joseph Griego
2022-03-25 09:30:21 PDT
Created attachment 455774 [details]
Patch
Created attachment 455798 [details]
Patch
update binding test expectations
Comment on attachment 455798 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455798&action=review > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:4010 > + return "([&] { auto* base = $globalObjectPtr; auto obj = jsDynamicCast<$wrapperType*>(base->vm(), base); return obj && $sideCondition; })()"; WebKit coding style uses words rather than non-word abbreviations. So this would be "object" rather than "obj". Same in the many other cases below and above. Thanks Darin, and apologies; I need to add a pre-commit hook for myself to yell when a variable is named `obj`! Old habits are hard to break Created attachment 457120 [details]
Patch
Created attachment 457810 [details]
Patch
Committed r293021 (249760@main): <https://commits.webkit.org/249760@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 457810 [details]. |