Chromium Code Reviews| Index: media/blink/webmediaplayer_impl.h |
| diff --git a/media/blink/webmediaplayer_impl.h b/media/blink/webmediaplayer_impl.h |
| index dd29a3380a0b3edde32f29e22d49d386656f2a6b..fe7132453c06d8bf04cefa42790a99d9119be5d5 100644 |
| --- a/media/blink/webmediaplayer_impl.h |
| +++ b/media/blink/webmediaplayer_impl.h |
| @@ -656,8 +656,15 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl |
| // Used for HLS playback and in certain fallback paths (e.g. on older devices |
| // that can't support the unified media pipeline). |
| - GURL fallback_url_; |
| - bool use_fallback_path_; |
| + GURL loaded_url_; |
| + |
| + // NOTE: |using_media_player_renderer_| is set based on the usage of a |
| + // MediaResource::Type::URL in StartPipeline(). This currently works because |
| + // the MediaPlayerRendererClient factory is the only factory that returns that |
| + // Type, but this may no longer be accurate when we remove |cast_impl_| and |
| + // WebMediaPlayerCast. This flag should be renamed/updated accordingly when |
| + // removing |cast_impl_|. |
| + bool using_media_player_renderer_; |
|
liberato (no reviews please)
2017/04/26 05:54:50
= false?
tguilbert
2017/04/26 18:11:53
Done.
|
| // Called sometime after the media is suspended in a playing state in |
| // OnFrameHidden(), causing the state to change to paused. |