Unify the error checking of crypto::Encryptor and add WARN_UNUSED_RESULT to prevent misuse.

crypto/encryptor.h

Index: crypto/encryptor.h
diff --git a/crypto/encryptor.h b/crypto/encryptor.h
index a40726673a43e5f02e1d2ef4f25fcb848a3341f7..0d52098b7d49e53f887701df275e45745c931f14 100644
--- a/crypto/encryptor.h
+++ b/crypto/encryptor.h
@@ -9,6 +9,7 @@
 #include <string>
 
 #include "base/basictypes.h"
+#include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string_piece.h"
 #include "build/build_config.h"
@@ -39,7 +40,7 @@ class CRYPTO_EXPORT Encryptor {
     ~Counter();
 
     // Increment the counter value.
-    bool Increment();
+    bool Increment() WARN_UNUSED_RESULT;
 
     // Write the content of the counter to |buf|. |buf| should have enough
     // space for |GetLengthInBytes()|.
@@ -62,19 +63,22 @@ class CRYPTO_EXPORT Encryptor {
   // key or the initialization vector cannot be used.
   //
   // When |mode| is CTR then |iv| should be empty.
-  bool Init(SymmetricKey* key, Mode mode, const base::StringPiece& iv);
+  bool Init(SymmetricKey* key, Mode mode,
+            const base::StringPiece& iv) WARN_UNUSED_RESULT;
 
   // Encrypts |plaintext| into |ciphertext|.
-  bool Encrypt(const base::StringPiece& plaintext, std::string* ciphertext);
+  bool Encrypt(const base::StringPiece& plaintext,
+               std::string* ciphertext) WARN_UNUSED_RESULT;
 
   // Decrypts |ciphertext| into |plaintext|.
-  bool Decrypt(const base::StringPiece& ciphertext, std::string* plaintext);
+  bool Decrypt(const base::StringPiece& ciphertext,
+               std::string* plaintext) WARN_UNUSED_RESULT;
 
   // Sets the counter value when in CTR mode. Currently only 128-bits
   // counter value is supported.
   //
   // Returns true only if update was successful.
-  bool SetCounter(const base::StringPiece& counter);
+  bool SetCounter(const base::StringPiece& counter) WARN_UNUSED_RESULT;
 
   // TODO(albertb): Support streaming encryption.