From  bug 631551 #c35:

When auto-submit is enabled, if there are unresolved comments (i.e. comments where the "Resolved" box isn't checked), when I click Code-Review +1, rather than auto-selecting Commit-Queue +2 (with the standard message about auto-submit), leave the Commit-Queue as 0 and instead display a message saying something like, "Auto-Submit suppressed due to unresolved comments. Choose CQ+2 (or mark comments as resolved) to trigger submit."

My reasoning:
- If I add comments to the CL, I very likely want the author to consider making edits. I probably don't want to trigger CQ+2 in that case.
  - If I only added an "FYI" comment and didn't intend for the author to make changes, I should have marked the comment as Resolved. I can also override and choose CQ+2 if I want.
  - I still want to mark the CL as Code-Review +1 to indicate that the CL is fine as-is if the author decides not to make changes based on my comments.
- Maybe it's just because I'm used to the workflow without auto-submit, but I very nearly hit "Send" on a CL where I made a comment before noticing that it was marked CQ+2 due to auto-submit.