[Webkit-unassigned] [Bug 190302] Expose whether WKWebProcessPlugInNodeHandle is a select element to clients

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 5 07:57:50 PDT 2018


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

--- Comment #8 from Zach Li <zacharyli323 at gmail.com> ---
(In reply to Tim Horton from comment #5)
> Comment on attachment 351653 [details]
> Patch v1
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=351653&action=review
> 
> >>> Tools/TestWebKitAPI/Tests/WebKitCocoa/InjectedBundleNodeHandleIsSelectElement.mm:45
> >>> +    WKDOMElement *htmlElement = [document createElement:htmlElementTag];
> >> 
> >> I don’t know this API well enough; what’s the ownership story for this object returned from a -create method?
> > 
> > `htmlElement` holds a reference to the object and once the method returns, nothing holds a reference to the object and the object gets deallocated.
> 
> No, that's not right, because this project doesn't build with ARC.

Oh, I didn't know that is the case.

Why is that?

> 
> However, I think the code is fine, because createElement returns an
> autoreleased object.

-- 
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/20181005/a9f7d3ad/attachment.html>


More information about the webkit-unassigned mailing list