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

Side by Side Diff: media/blink/webmediaplayer_impl.h

Issue 2276033002: Pass SkPaint instead of its alpha and mode in WebMediaPlayer::paint (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update layout expectation Created 4 years, 3 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 MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 5 #ifndef MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 void setBufferingStrategy( 115 void setBufferingStrategy(
116 blink::WebMediaPlayer::BufferingStrategy buffering_strategy) override; 116 blink::WebMediaPlayer::BufferingStrategy buffering_strategy) override;
117 blink::WebTimeRanges buffered() const override; 117 blink::WebTimeRanges buffered() const override;
118 blink::WebTimeRanges seekable() const override; 118 blink::WebTimeRanges seekable() const override;
119 119
120 // paint() the current video frame into |canvas|. This is used to support 120 // paint() the current video frame into |canvas|. This is used to support
121 // various APIs and functionalities, including but not limited to: <canvas>, 121 // various APIs and functionalities, including but not limited to: <canvas>,
122 // WebGL texImage2D, ImageBitmap, printing and capturing capabilities. 122 // WebGL texImage2D, ImageBitmap, printing and capturing capabilities.
123 void paint(blink::WebCanvas* canvas, 123 void paint(blink::WebCanvas* canvas,
124 const blink::WebRect& rect, 124 const blink::WebRect& rect,
125 unsigned char alpha, 125 const SkPaint* paint) override;
126 SkXfermode::Mode mode) override;
127 126
128 // True if the loaded media has a playable video/audio track. 127 // True if the loaded media has a playable video/audio track.
129 bool hasVideo() const override; 128 bool hasVideo() const override;
130 bool hasAudio() const override; 129 bool hasAudio() const override;
131 130
132 void enabledAudioTracksChanged( 131 void enabledAudioTracksChanged(
133 const blink::WebVector<blink::WebMediaPlayer::TrackId>& enabledTrackIds) 132 const blink::WebVector<blink::WebMediaPlayer::TrackId>& enabledTrackIds)
134 override; 133 override;
135 void selectedVideoTrackChanged( 134 void selectedVideoTrackChanged(
136 blink::WebMediaPlayer::TrackId* selectedTrackId) override; 135 blink::WebMediaPlayer::TrackId* selectedTrackId) override;
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
540 // Monitors the watch time of the played content. 539 // Monitors the watch time of the played content.
541 std::unique_ptr<WatchTimeReporter> watch_time_reporter_; 540 std::unique_ptr<WatchTimeReporter> watch_time_reporter_;
542 bool is_encrypted_; 541 bool is_encrypted_;
543 542
544 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl); 543 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
545 }; 544 };
546 545
547 } // namespace media 546 } // namespace media
548 547
549 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 548 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698