Drop DecryptConfig::data_offset_.

media/webm/webm_crypto_helpers.cc

Index: media/webm/webm_crypto_helpers.cc
diff --git a/media/webm/webm_crypto_helpers.cc b/media/webm/webm_crypto_helpers.cc
index a663f3cd6a6a2b318108328c5ef42249c3abf4cf..d6af43f4d377679a7158235e71f3f405be7f6cb1 100644
--- a/media/webm/webm_crypto_helpers.cc
+++ b/media/webm/webm_crypto_helpers.cc
@@ -24,12 +24,13 @@ std::string GenerateWebMCounterBlock(const uint8* iv, int iv_size) {
 
 }  // namespace anonymous
 
-scoped_ptr<DecryptConfig> WebMCreateDecryptConfig(
-    const uint8* data, int data_size,
-    const uint8* key_id, int key_id_size) {
+bool WebMCreateDecryptConfig(const uint8* data, int data_size,
+                             const uint8* key_id, int key_id_size,
+                             scoped_ptr<DecryptConfig>* decrypt_config,

[DaleCurtis, 2014/01/02 19:31:09]

Just make it a regular pointer? This method shouldn't really care how the caller
allocates the DecryptConfig.

[xhwang, 2014/01/08 00:43:36]

Hmm, we have "new" on l.53. If we return a regular pointer, the caller has to
remember to "delete" it, or wrap it in a scoped_ptr<>. I like to wrap it
immediately so that the ownership is super clear. Maybe I missed something?

+                             int* data_offset) {
   if (data_size < kWebMSignalByteSize) {
     DVLOG(1) << "Got a block from an encrypted stream with no data.";
-    return scoped_ptr<DecryptConfig>();
+    return false;
   }
 
   uint8 signal_byte = data[0];
@@ -43,18 +44,19 @@ scoped_ptr<DecryptConfig> WebMCreateDecryptConfig(
   if (signal_byte & kWebMFlagEncryptedFrame) {
     if (data_size < kWebMSignalByteSize + kWebMIvSize) {
       DVLOG(1) << "Got an encrypted block with not enough data " << data_size;
-      return scoped_ptr<DecryptConfig>();
+      return false;
     }
     counter_block = GenerateWebMCounterBlock(data + frame_offset, kWebMIvSize);
     frame_offset += kWebMIvSize;
   }
 
-  scoped_ptr<DecryptConfig> config(new DecryptConfig(
+  decrypt_config->reset(new DecryptConfig(
       std::string(reinterpret_cast<const char*>(key_id), key_id_size),
       counter_block,
-      frame_offset,
       std::vector<SubsampleEntry>()));
-  return config.Pass();
+  *data_offset = frame_offset;
+
+  return true;
 }
 
 }  // namespace media