Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(60)

Side by Side Diff: content/renderer/media/webmediaplayer_ms.h

Issue 2490783002: Refactor WebMediaPlayerDelegate interface. (Closed)
Patch Set: Fix did_play_ to only be set once per play. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_
6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_ 6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 bool didPassCORSAccessCheck() const override; 136 bool didPassCORSAccessCheck() const override;
137 137
138 double mediaTimeForTimeValue(double timeValue) const override; 138 double mediaTimeForTimeValue(double timeValue) const override;
139 139
140 unsigned decodedFrameCount() const override; 140 unsigned decodedFrameCount() const override;
141 unsigned droppedFrameCount() const override; 141 unsigned droppedFrameCount() const override;
142 size_t audioDecodedByteCount() const override; 142 size_t audioDecodedByteCount() const override;
143 size_t videoDecodedByteCount() const override; 143 size_t videoDecodedByteCount() const override;
144 144
145 // WebMediaPlayerDelegate::Observer implementation. 145 // WebMediaPlayerDelegate::Observer implementation.
146 void OnHidden() override; 146 void OnFrameHidden() override;
147 void OnShown() override; 147 void OnFrameClosed() override;
148 bool OnSuspendRequested(bool must_suspend) override; 148 void OnFrameShown() override;
149 void OnIdleTimeout() override;
149 void OnPlay() override; 150 void OnPlay() override;
150 void OnPause() override; 151 void OnPause() override;
151 void OnVolumeMultiplierUpdate(double multiplier) override; 152 void OnVolumeMultiplierUpdate(double multiplier) override;
152 153
153 bool copyVideoTextureToPlatformTexture(gpu::gles2::GLES2Interface* gl, 154 bool copyVideoTextureToPlatformTexture(gpu::gles2::GLES2Interface* gl,
154 unsigned int texture, 155 unsigned int texture,
155 unsigned int internal_format, 156 unsigned int internal_format,
156 unsigned int type, 157 unsigned int type,
157 bool premultiply_alpha, 158 bool premultiply_alpha,
158 bool flip_y) override; 159 bool flip_y) override;
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 // True if playback should be started upon the next call to OnShown(). Only 252 // True if playback should be started upon the next call to OnShown(). Only
252 // used on Android. 253 // used on Android.
253 bool should_play_upon_shown_; 254 bool should_play_upon_shown_;
254 255
255 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerMS); 256 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerMS);
256 }; 257 };
257 258
258 } // namespace content 259 } // namespace content
259 260
260 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_ 261 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698