| commit | 13d0074ae638f5674c784e6568bacf5102d1df98 | [log] [tgz] |
|---|---|---|
| author | vabr <[email protected]> | Mon Jun 05 10:31:46 2017 |
| committer | Commit Bot <[email protected]> | Mon Jun 05 10:31:46 2017 |
| tree | cf1bbaf62cc0e94bb72998bf8ee8eb4c50c2675d | |
| parent | 0afe568a073a2394d826c084f1878fd0ef19bd49 [diff] [blame] |
Don't move raw pointers in net Moving raw pointers does the same job as copying, but is more verbose and also more confusing: e.g., is the supposed pointer meant to be a smart one? This instance was flagged by the tool from https://codereview.chromium.org/2919243002/. BUG=729393 Review-Url: https://codereview.chromium.org/2922883002 Cr-Commit-Position: refs/heads/master@{#476968}
diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc index 3dded3d..fc429bc 100644 --- a/net/url_request/url_request_unittest.cc +++ b/net/url_request/url_request_unittest.cc
@@ -6782,7 +6782,7 @@ context.set_network_delegate(&network_delegate); context.set_cert_verifier(&cert_verifier); context.set_cert_transparency_verifier(&ct_verifier); - context.set_ct_policy_enforcer(std::move(&ct_policy_enforcer)); + context.set_ct_policy_enforcer(&ct_policy_enforcer); context.Init(); // Now send a request to trigger the violation.