Improve documentation of CertVerifier::Verify() regarding |out_req|.

Improve documentation of CertVerifier::Verify() regarding |out_req|.

Also adds DISALLOW_COPY() to Request.

BUG=475153
Committed: https://crrev.com/9ef2155288547276afebfed59fc144402c286370
Cr-Commit-Position: refs/heads/master@{#330840}

[eroman, 2015-05-15 03:05:41]

eroman@chromium.org changed reviewers:
+ pneubeck@chromium.org

[pneubeck (no reviews), 2015-05-15 09:19:35]

On 2015/05/15 03:05:42, eroman (sick slow) wrote:

Sorry, I think my original comment was written a bit to hasty.
To prevent the copy this also requires adding a default constructor, otherwise
it doesn't compile anymore (I ran a try-bot to verify that):

class Request {
 public:
  Request() {}

  // Destruction of the Request cancels it.
  virtual ~Request() {}

 private:
  DISALLOW_COPY_AND_ASSIGN(Request);
};

should work.

Since the class is not abstract, assignment (being a no-op) would also be
possible if not disallowed.
As the overhead is negligible you could disallow it as well...

[eroman, 2015-05-19 19:43:14]

Done, thanks

[pneubeck (no reviews), 2015-05-20 07:56:30]

lgtm

[eroman, 2015-05-20 21:41:26]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2015-05-20 21:42:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1131173005/60001

[commit-bot: I haz the power, 2015-05-20 23:33:11]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-20 23:34:33]

Patchset 4 (id:??) landed as
https://crrev.com/9ef2155288547276afebfed59fc144402c286370
Cr-Commit-Position: refs/heads/master@{#330840}