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

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

Issue 2490783002: Refactor WebMediaPlayerDelegate interface. (Closed)
Patch Set: Unit tests found a real bug! 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 bool premultiply_alpha, 156 bool premultiply_alpha,
156 bool flip_y) override; 157 bool flip_y) override;
157 158
158 bool texImageImpl(TexImageFunctionID functionID, 159 bool texImageImpl(TexImageFunctionID functionID,
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 // True if playback should be started upon the next call to OnShown(). Only 250 // True if playback should be started upon the next call to OnShown(). Only
250 // used on Android. 251 // used on Android.
251 bool should_play_upon_shown_; 252 bool should_play_upon_shown_;
252 253
253 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerMS); 254 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerMS);
254 }; 255 };
255 256
256 } // namespace content 257 } // namespace content
257 258
258 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_ 259 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_MS_H_
OLDNEW
« no previous file with comments | « content/renderer/media/renderer_webmediaplayer_delegate_browsertest.cc ('k') | content/renderer/media/webmediaplayer_ms.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698