Refactored DecoderBuffer to use unix_hacker_style naming.

media/base/decoder_buffer.h

Index: media/base/decoder_buffer.h
diff --git a/media/base/decoder_buffer.h b/media/base/decoder_buffer.h
index d1c20ddda362673e2c5c191d0844c4714814cee7..a116da6a0c437c46d98b327e828b22277b3d8356 100644
--- a/media/base/decoder_buffer.h
+++ b/media/base/decoder_buffer.h
@@ -26,7 +26,7 @@ class DecryptConfig;
 //
 // Also includes decoder specific functionality for decryption.
 //
-// NOTE: It is illegal to call any method when IsEndOfStream() is true.
+// NOTE: It is illegal to call any method when is_end_of_stream() is true.
 class MEDIA_EXPORT DecoderBuffer
     : public base::RefCountedThreadSafe<DecoderBuffer> {
  public:
@@ -45,43 +45,43 @@ class MEDIA_EXPORT DecoderBuffer
 
   // Create a DecoderBuffer whose |data_| is copied from |data|.  Buffer will be
   // padded and aligned as necessary.  |data| must not be NULL and |size| >= 0.
-  static scoped_refptr<DecoderBuffer> CopyFrom(const uint8* data, int size);
+  static scoped_refptr<DecoderBuffer> copy_from(const uint8* data, int size);

[scherkus (not reviewing), 2013/06/20 21:26:59]

whoops! you changed too much :(

style guide states only simple getters/setters (like the ones below) get
unix_hacker style

complex methods are CamelCase (e.g., CopyFrom(), AsHumanReadableString())

 
   // Create a DecoderBuffer whose |data_| is copied from |data| and |side_data_|
   // is copied from |side_data|. Buffers will be padded and aligned as necessary
   // Data pointers must not be NULL and sizes must be >= 0.
-  static scoped_refptr<DecoderBuffer> CopyFrom(const uint8* data, int size,
+  static scoped_refptr<DecoderBuffer> copy_from(const uint8* data, int size,
                                                const uint8* side_data,
                                                int side_data_size);
 
   // Create a DecoderBuffer indicating we've reached end of stream.
   //
-  // Calling any method other than IsEndOfStream() on the resulting buffer
+  // Calling any method other than is_end_of_stream() on the resulting buffer
   // is disallowed.
-  static scoped_refptr<DecoderBuffer> CreateEOSBuffer();
+  static scoped_refptr<DecoderBuffer> create_eos_buffer();

[scherkus (not reviewing), 2013/06/20 21:26:59]

CreateEOSBuffer()

 
-  base::TimeDelta GetTimestamp() const;
-  void SetTimestamp(const base::TimeDelta& timestamp);
+  base::TimeDelta get_timestamp() const;

[scherkus (not reviewing), 2013/06/20 21:26:59]

with unix_hacker style getters don't feature the "get_" prefix

can you change this to:
timestamp()
set_timestamp(...)

as well as the rest below

+  void set_timestamp(const base::TimeDelta& timestamp);
 
-  base::TimeDelta GetDuration() const;
-  void SetDuration(const base::TimeDelta& duration);
+  base::TimeDelta get_duration() const;
+  void set_duration(const base::TimeDelta& duration);
 
-  const uint8* GetData() const;
-  uint8* GetWritableData() const;
+  const uint8* get_data() const;
+  uint8* get_writable_data() const;
 
-  int GetDataSize() const;
+  int get_data_size() const;
 
-  const uint8* GetSideData() const;
-  int GetSideDataSize() const;
+  const uint8* get_side_data() const;
+  int get_side_data_size() const;
 
-  const DecryptConfig* GetDecryptConfig() const;
+  const DecryptConfig* get_decrypt_config() const;
   void SetDecryptConfig(scoped_ptr<DecryptConfig> decrypt_config);
 
   // If there's no data in this buffer, it represents end of stream.
-  bool IsEndOfStream() const;
+  bool is_end_of_stream() const;

[scherkus (not reviewing), 2013/06/20 21:26:59]

similar story with bools getters where we don't feature an "is_" prefix:

bool end_of_stream() const;

 
   // Returns a human-readable string describing |*this|.
-  std::string AsHumanReadableString();
+  std::string as_human_readable_string();

[scherkus (not reviewing), 2013/06/20 21:26:59]

revert this change

 
  protected:
   friend class base::RefCountedThreadSafe<DecoderBuffer>;