Add missing return statement to CheckReturnValue's copy assignment operator.

Add missing return statement to CheckReturnValue's copy assignment operator.

BUG=none
R=huangs@chromium.org

Committed: https://crrev.com/475681390eea963b4bbba2308d7aff4383254904
Cr-Commit-Position: refs/heads/master@{#433896}

[grt (UTC plus 2), 2016-10-26 11:38:41]

The CQ bit was checked by grt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 11:38:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[grt (UTC plus 2), 2016-10-26 11:41:55]

Please check the CQ box if this looks good to you. Thanks.

[commit-bot: I haz the power, 2016-10-26 12:36:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 12:36:15]

Dry run: This issue passed the CQ dry run.

[grt (UTC plus 2), 2016-11-22 07:37:01]

The CQ bit was checked by grt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 07:37:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[grt (UTC plus 2), 2016-11-22 07:37:27]

huangs: ping! please check the cq box if it looks good to you. thanks.

[commit-bot: I haz the power, 2016-11-22 08:18:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-22 08:18:57]

Dry run: This issue passed the CQ dry run.

[huangs, 2016-11-22 17:12:48]

Really sorry, didn't see this!  LGTM.

Interesting that unused template functions with missing return value don't
generate an error. Did you find this using a linter or a different compiler?

[huangs, 2016-11-22 17:12:54]

The CQ bit was checked by huangs@chromium.org

[commit-bot: I haz the power, 2016-11-22 17:13:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 17:17:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479834774312690,
"parent_rev": "7fa39f2b05c17b538280f85516ebbb02f7d7a921", "commit_rev":
"71edc6034688b5325c86fa3ca5cdf7815ad13772"}

[commit-bot: I haz the power, 2016-11-22 17:17:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-22 17:23:39]

Description was changed from

==========
Add missing return statement to CheckReturnValue's copy assignment operator.

BUG=none
R=huangs@chromium.org
==========

to

==========
Add missing return statement to CheckReturnValue's copy assignment operator.

BUG=none
R=huangs@chromium.org

Committed: https://crrev.com/475681390eea963b4bbba2308d7aff4383254904
Cr-Commit-Position: refs/heads/master@{#433896}
==========

[commit-bot: I haz the power, 2016-11-22 17:23:41]

Patchset 2 (id:??) landed as
https://crrev.com/475681390eea963b4bbba2308d7aff4383254904
Cr-Commit-Position: refs/heads/master@{#433896}