Improve retry support for media network loading.

media/blink/buffered_data_source.h

Index: media/blink/buffered_data_source.h
diff --git a/media/blink/buffered_data_source.h b/media/blink/buffered_data_source.h
index 17b7507bd93133d8bdbd2d42441da644132ca466..ff2c55deca54524009abc7398b493bb2e1e4cb0a 100644
--- a/media/blink/buffered_data_source.h
+++ b/media/blink/buffered_data_source.h
@@ -122,6 +122,8 @@ class BufferedDataSourceInterface : public DataSource {
 class MEDIA_BLINK_EXPORT BufferedDataSource
     : NON_EXPORTED_BASE(public BufferedDataSourceInterface) {
  public:
+  // Number of times we retry before erroring out.

[DaleCurtis, 2016/03/11 16:36:32]

Static const on an interface is odd. Enum is preferred these days.  On mobile,
but can send discussion later. You can add the other retry const here too then
and avoid duplicate defs.

[hubbe, 2016/03/11 18:15:55]

This is not an interface, but regardless, enums preferred.
(I remember reading the discussion, after you pointed it out..)

Since intend to delete this class later, I don't want to de-dupe the
two constants.

+  static const int kLoaderRetries;
   typedef base::Callback<void(bool)> DownloadingCB;
 
   // |url| and |cors_mode| are passed to the object. Buffered byte range changes