Skip to content

Commit

Permalink
Remove unused Paint() method in WebMediaPlayer.
Browse files Browse the repository at this point in the history
The signiture of this method is changed, so old Paint() method is removed.

BUG=407079

Review URL: https://codereview.chromium.org/650943002

Cr-Commit-Position: refs/heads/master@{#299323}
  • Loading branch information
ds-hwang authored and Commit bot committed Oct 13, 2014
1 parent 77c1403 commit 2575741
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 30 deletions.
6 changes: 0 additions & 6 deletions content/renderer/media/android/webmediaplayer_android.cc
Original file line number Diff line number Diff line change
Expand Up @@ -574,12 +574,6 @@ bool WebMediaPlayerAndroid::EnsureTextureBackedSkBitmap(GrContext* gr,
return true;
}

void WebMediaPlayerAndroid::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha) {
paint(canvas, rect, alpha, SkXfermode::kSrcOver_Mode);
}

void WebMediaPlayerAndroid::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha,
Expand Down
4 changes: 0 additions & 4 deletions content/renderer/media/android/webmediaplayer_android.h
Original file line number Diff line number Diff line change
Expand Up @@ -126,10 +126,6 @@ class WebMediaPlayerAndroid : public blink::WebMediaPlayer,
const blink::WebRect& rect,
unsigned char alpha,
SkXfermode::Mode mode);
// TODO(dshwang): remove it because above method replaces. crbug.com/401027
virtual void paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha);

virtual bool copyVideoTextureToPlatformTexture(
blink::WebGraphicsContext3D* web_graphics_context,
Expand Down
6 changes: 0 additions & 6 deletions content/renderer/media/webmediaplayer_ms.cc
Original file line number Diff line number Diff line change
Expand Up @@ -319,12 +319,6 @@ bool WebMediaPlayerMS::didLoadingProgress() {
return true;
}

void WebMediaPlayerMS::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha) {
paint(canvas, rect, alpha, SkXfermode::kSrcOver_Mode);
}

void WebMediaPlayerMS::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha,
Expand Down
4 changes: 0 additions & 4 deletions content/renderer/media/webmediaplayer_ms.h
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,6 @@ class WebMediaPlayerMS
const blink::WebRect& rect,
unsigned char alpha,
SkXfermode::Mode mode);
// TODO(dshwang): remove it because above method replaces. crbug.com/401027
virtual void paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha);

// True if the loaded media has a playable video/audio track.
virtual bool hasVideo() const;
Expand Down
6 changes: 0 additions & 6 deletions media/blink/webmediaplayer_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -507,12 +507,6 @@ bool WebMediaPlayerImpl::didLoadingProgress() {
return pipeline_progress || data_progress;
}

void WebMediaPlayerImpl::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha) {
paint(canvas, rect, alpha, SkXfermode::kSrcOver_Mode);
}

void WebMediaPlayerImpl::paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha,
Expand Down
4 changes: 0 additions & 4 deletions media/blink/webmediaplayer_impl.h
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,6 @@ class MEDIA_EXPORT WebMediaPlayerImpl
const blink::WebRect& rect,
unsigned char alpha,
SkXfermode::Mode mode);
// TODO(dshwang): remove it because above method replaces. crbug.com/401027
virtual void paint(blink::WebCanvas* canvas,
const blink::WebRect& rect,
unsigned char alpha);

// True if the loaded media has a playable video/audio track.
virtual bool hasVideo() const;
Expand Down

0 comments on commit 2575741

Please sign in to comment.