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

Unified Diff: cc/resources/video_resource_updater.h

Issue 2763223003: Reland of cc: Don't use StreamVideoDrawQuad on any platform but Android. (Closed)
Patch Set: Address Dana's comments. Fix unittest. Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: cc/resources/video_resource_updater.h
diff --git a/cc/resources/video_resource_updater.h b/cc/resources/video_resource_updater.h
index 7cc2c813597913a7259f16b34060a0a04b1e18e6..9bf52ac8588920b2d95c98cf66b0cc1972008290 100644
--- a/cc/resources/video_resource_updater.h
+++ b/cc/resources/video_resource_updater.h
@@ -74,7 +74,8 @@ class CC_EXPORT VideoResourceUpdater
: public base::SupportsWeakPtr<VideoResourceUpdater> {
public:
VideoResourceUpdater(ContextProvider* context_provider,
- ResourceProvider* resource_provider);
+ ResourceProvider* resource_provider,
+ bool use_stream_video_draw_quad = false);
danakj 2017/03/28 15:44:13 i would prefer not to use a default value here, it
Daniele Castagna 2017/03/29 20:38:18 Done.
~VideoResourceUpdater();
VideoFrameExternalResources CreateExternalResourcesFromVideoFrame(
@@ -198,6 +199,7 @@ class CC_EXPORT VideoResourceUpdater
ContextProvider* context_provider_;
ResourceProvider* resource_provider_;
+ const bool use_stream_video_draw_quad_;
std::unique_ptr<media::SkCanvasVideoRenderer> video_renderer_;
std::vector<uint8_t> upload_pixels_;

Powered by Google App Engine
This is Rietveld 408576698