Bug 121453 – Replace more uses of PassOwnPtr with OwnPtr in WebKit2

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Anders Carlsson]

Replace more uses of PassOwnPtr with OwnPtr in WebKit2

[Anders Carlsson]

Created attachment 211833 [details]
Patch

[Darin Adler]

Comment on attachment 211833 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=211833&action=review

> Source/WebKit2/Platform/CoreIPC/Connection.cpp:100
>  WTF_MAKE_NONCOPYABLE(SecondaryThreadPendingSyncReply);

I think that having an OwnPtr will automatically make the class noncopyable, so you could just omit this now.

> Source/WebKit2/Platform/CoreIPC/Connection.cpp:102
> -    SecondaryThreadPendingSyncReply() : replyDecoder(0) { }
> +    SecondaryThreadPendingSyncReply() { }

Should just omit this entirely and let the compiler generate it.

> Source/WebKit2/Platform/CoreIPC/Connection.cpp:640
> +    const auto it = m_workQueueMessageReceivers.find(message->messageReceiverName());

Why const?

> Source/WebKit2/Platform/CoreIPC/Connection.cpp:656
> +        auto it = m_waitForMessageMap.find(std::make_pair(std::make_pair(message->messageReceiverName(), message->messageName()), message->destinationID()));

You didn’t add const here, so why did you add it above?

[Anders Carlsson]

Comment on attachment 211833 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=211833&action=review

>> Source/WebKit2/Platform/CoreIPC/Connection.cpp:102
>> +    SecondaryThreadPendingSyncReply() { }
> 
> Should just omit this entirely and let the compiler generate it.

I tried doing that but it wouldn’t work due to the WTF_MAKE_NONCOPYABLE. I’ve removed both now.

>> Source/WebKit2/Platform/CoreIPC/Connection.cpp:640
>> +    const auto it = m_workQueueMessageReceivers.find(message->messageReceiverName());
> 
> Why const?

No good reason, I’ll remove it.

[Anders Carlsson]

Committed r155905: <http://trac.webkit.org/changeset/155905>