-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Generate serialization for NSPresentationIntent #24035
Generate serialization for NSPresentationIntent #24035
Conversation
result = [NSPresentationIntent thematicBreakIntentWithIdentity:m_identity nestedInsideIntent:parent]; | ||
break; | ||
case NSPresentationIntentKindTable: | ||
columnAlignments = [[NSMutableArray alloc] initWithCapacity:m_columnAlignments.size()]; |
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.
This is a memory leak.
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.
It's not, because of the [columnAlignments release]
that was further down, but I've changed to use smart pointers to be consistent with The WK Wayβ’.
c74037b
to
7abaabc
Compare
7abaabc
to
4cba283
Compare
|
||
int64_t m_intentKind { 0 }; | ||
int64_t m_identity { 0 }; | ||
std::unique_ptr<CoreIPCPresentationIntent> m_parentIntent { nullptr }; |
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.
std::unique_ptr has a default constructor that initializes the pointer, so { nullptr } is unnecessary.
|
||
#if PLATFORM(COCOA) | ||
|
||
#include "ArgumentCodersCocoa.h" |
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.
This also seems unnecessary. <wtf/ArgumentCoder.h> should be sufficient.
EWS run on previous version of this PR (hash c74037b) |
EWS run on previous version of this PR (hash 7abaabc) |
4cba283
to
1dd1fd2
Compare
EWS run on previous version of this PR (hash 4cba283) |
EWS run on current version of this PR (hash 1dd1fd2) |
https://bugs.webkit.org/show_bug.cgi?id=268779 rdar://109121874 Reviewed by achristensen07 (Alex Christensen). * Source/WebKit/DerivedSources-input.xcfilelist: * Source/WebKit/DerivedSources.make: * Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.h: * Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.mm: (IPC::typeFromObject): (IPC::shouldEnableStrictMode): * Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.h: * Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.mm: (WebKit::valueFromID): * Source/WebKit/Shared/Cocoa/CoreIPCPresentationIntent.h: Added. (WebKit::CoreIPCPresentationIntent::CoreIPCPresentationIntent): * Source/WebKit/Shared/Cocoa/CoreIPCPresentationIntent.mm: Added. (WebKit::CoreIPCPresentationIntent::CoreIPCPresentationIntent): (WebKit::CoreIPCPresentationIntent::toID const): * Source/WebKit/Shared/Cocoa/CoreIPCPresentationIntent.serialization.in: Added. * Source/WebKit/Shared/Cocoa/CoreIPCTypes.h: * Source/WebKit/WebKit.xcodeproj/project.pbxproj: * Tools/TestWebKitAPI/Tests/IPC/IPCSerialization.mm: (TEST): Canonical link: https://commits.webkit.org/274269@main
1dd1fd2
to
089245f
Compare
Committed 274269@main (089245f): https://commits.webkit.org/274269@main Reviewed commits have been landed. Closing PR #24035 and removing active labels. |
089245f
1dd1fd2
π wincairoπ§ͺ wpe-wk2π§ͺ api-macπ§ͺ ios-wk2-wptπ§ͺ mac-wk2π tvπ watch