Bug 198214 – Update messages_unittest.py after r245715

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[youenn fablet]

Update messages_unittest.py after https://trac.webkit.org/changeset/245715 to cope with messages.py changes.

[youenn fablet]

Created attachment 370563 [details]
Patch

[Alexey Proskuryakov]

Comment on attachment 370563 [details]
Patch

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

> Source/WebKit/ChangeLog:3
> +        Update messages_unittest.py after https://trac.webkit.org/changeset/245715

Probably enough to say r245715 here.

> Source/WebKit/ChangeLog:9
> +        Small update to messages.py so that cancelReply is called in case of decoding error.

The patch looks good overall, but the ChangeLog confuses me. Isn't the change to messages.py more important? I know that we don't expect this path to be taken, but it has some potential for changing observed behavior anyway. If that happens, everyone will be confused by the title.

[youenn fablet]

(In reply to Alexey Proskuryakov from comment #2)
> Comment on attachment 370563 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=370563&action=review
> 
> > Source/WebKit/ChangeLog:3
> > +        Update messages_unittest.py after https://trac.webkit.org/changeset/245715
> 
> Probably enough to say r245715 here.

Will do.

> > Source/WebKit/ChangeLog:9
> > +        Small update to messages.py so that cancelReply is called in case of decoding error.
> 
> The patch looks good overall, but the ChangeLog confuses me. Isn't the
> change to messages.py more important? I know that we don't expect this path
> to be taken, but it has some potential for changing observed behavior
> anyway. If that happens, everyone will be confused by the title.

Will update the changelog.
This patch does not change observed behavior though, r245715 does.

The changes here to messages.py are just to call cancelReply in case of decode error instead of calling completionHandler(IPC::AsyncReplyError<>...) as done in r245715.

[youenn fablet]

Created attachment 370577 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 370577 [details]
Patch for landing

Clearing flags on attachment: 370577

Committed r245749: <https://trac.webkit.org/changeset/245749>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/51115730>

[Alexey Proskuryakov]

> The changes here to messages.py are just to call cancelReply in case of decode error instead of calling completionHandler(IPC::AsyncReplyError<>...) as done in r245715.

Which is a change in behavior.