Bug 255675 – [REGRESSION] Reverting a patch no longer posts anything in the original bug

• Component: Tools / Tests
• Status: NEW

[Myles C. Maxfield]

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

This patch was reverted, but Fuji manually commented in it and reopened it themself. WebKitBot didn't comment in the bug _at all_.

When WebKitBot reverts a patch, it should _always_ comment in the bug of the patch it reverted.

[Radar WebKit Bug Importer]

<rdar://problem/108277861>

[[e-mail]]

I don't think webkitbot reopens a bug. I see cases where bug is reopened by Commit Bot.

webkitbot uses the "create-revert" option of webkit-patch consisting of the following steps:

        steps.DiscardLocalChanges,
        steps.Update,
        steps.RevertRevision,
        steps.CreateBug,
        steps.PrepareChangeLogForRevert,
        steps.PostDiffForRevert,

There is another webkit-patch option "revert" which consist of 
        steps.DiscardLocalChanges,
        steps.Update,
        steps.RevertRevision,
        steps.PrepareChangeLogForRevert,
        steps.EditChangeLog,
        steps.ConfirmDiff,
        steps.Build,
        steps.Commit,
        steps.ReopenBugAfterRevert,

[Alexey Proskuryakov]

This is generally happening in practice though.

https://bugs.webkit.org/show_bug.cgi?id=255610 was a revert bug created by webkitbot. It was filed at 10:02:18 PDT, and https://bugs.webkit.org/show_bug.cgi?id=255506 got reopened three seconds later. Commit queue only landed the revert at 10:15:04 PDT.

Looking at the original case that motivated this report, https://bugs.webkit.org/show_bug.cgi?id=255473, I see that Fujii reopened it manually BEFORE creating the revert bug. Webkitbot did add a dependency relationship.

So, seems like correct behavior - it would be wrong for webkitbot to say that it reopened the bug when it didn't. Remember that the comment says "Re-opened since this is blocked by <...>". Myles, is your expectation different?

> When WebKitBot reverts a patch, it should _always_ comment in the bug of the patch it reverted.

Note however that webkitbot does not comment after reverting, it only ever comments when creating a revert bug. I think that this is wrong, both commenting and reopening should happen when the revert lands. But maybe I'm missing why it's being done this way. In any case, there is no regression here, and we'd want a new bug report to investigate improvements.