Fix base::StringPiece calling conventions in //crypto.

Per the documentation in the header, base::StringPiece should be passed
by value, not by const reference.

Bug: 
Change-Id: I21351fcb901f5ef89fa7b84af0721d483b333810
Reviewed-on: https://chromium-review.googlesource.com/754584
Reviewed-by: Ryan Sleevi <[email protected]>
Commit-Queue: David Benjamin <[email protected]>
Cr-Commit-Position: refs/heads/master@{#514181}
diff --git a/crypto/sha2.h b/crypto/sha2.h
index d575815..f41224ee 100644
--- a/crypto/sha2.h
+++ b/crypto/sha2.h
@@ -23,12 +23,13 @@
 // Computes the SHA-256 hash of the input string 'str' and stores the first
 // 'len' bytes of the hash in the output buffer 'output'.  If 'len' > 32,
 // only 32 bytes (the full hash) are stored in the 'output' buffer.
-CRYPTO_EXPORT void SHA256HashString(const base::StringPiece& str,
-                                    void* output, size_t len);
+CRYPTO_EXPORT void SHA256HashString(base::StringPiece str,
+                                    void* output,
+                                    size_t len);
 
 // Convenience version of the above that returns the result in a 32-byte
 // string.
-CRYPTO_EXPORT std::string SHA256HashString(const base::StringPiece& str);
+CRYPTO_EXPORT std::string SHA256HashString(base::StringPiece str);
 
 }  // namespace crypto