New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add inspectable property to _WKWebExtensionContext. #7497
Conversation
EWS run on current version of this PR (hash 8befe62) |
@discussion A context can control multiple `WKWebView` instances, from the background content, to the popover. | ||
You should set this to `YES` when needed for debugging purposes. The default value is `NO`. | ||
*/ | ||
@property (nonatomic, getter=isInspectable) BOOL inspectable; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this property expose itself in Swift as isInspectable
, or do we need to add NS_SWIFT_NAME(isInspectable)
to match the inspectable
property from JSContext
and WKWebView
? (During API review we had determined the inspectable
was the correct property name for ObjC, and isInspectable
was the correct property name for Swift).
https://bugs.webkit.org/show_bug.cgi?id=249158 Reviewed by Brian Weinstein. Also fix some typos in other HeadDoc comments. * Source/WebKit/UIProcess/API/Cocoa/_WKWebExtension.h: * Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionContext.h: * Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionContext.mm: (-[_WKWebExtensionContext isInspectable]): Added. (-[_WKWebExtensionContext setInspectable:]): Added. * Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionContextCocoa.mm: (WebKit::WebExtensionContext::setInspectable): Added. (WebKit::WebExtensionContext::loadBackgroundWebView): Set inspectable proeprty. * Source/WebKit/UIProcess/Extensions/WebExtensionContext.h: (WebKit::WebExtensionContext::isInspectable const): Added. Canonical link: https://commits.webkit.org/257740@main
8befe62
to
aeaa834
Compare
Committed 257740@main (aeaa834): https://commits.webkit.org/257740@main Reviewed commits have been landed. Closing PR #7497 and removing active labels. |
aeaa834
8befe62
π mac-AS-debugπ gtkπ wincairoπ§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk1π§ͺ api-gtkπ tvπ§ͺ mac-wk2π§ͺ mac-AS-debug-wk2π watchπ§ͺ mac-wk2-stressπ watch-sim