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

Unified Diff: webkit/media/webmediaplayer_impl.cc

Issue 10837118: Dead code elimination: scythe.chrome_functions:segment.path %media% edition, round 1. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 4 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: webkit/media/webmediaplayer_impl.cc
diff --git a/webkit/media/webmediaplayer_impl.cc b/webkit/media/webmediaplayer_impl.cc
index 22a1476010dd71671018233f564a36476fd89b24..c5d6579beb61ece4d0b61ae48e2ff171b92d5c4e 100644
--- a/webkit/media/webmediaplayer_impl.cc
+++ b/webkit/media/webmediaplayer_impl.cc
@@ -513,13 +513,6 @@ void WebMediaPlayerImpl::setSize(const WebSize& size) {
// Don't need to do anything as we use the dimensions passed in via paint().
}
-// This variant (without alpha) is just present during staging of this API
-// change. Later we will again only have one virtual paint().
-void WebMediaPlayerImpl::paint(WebKit::WebCanvas* canvas,
- const WebKit::WebRect& rect) {
- paint(canvas, rect, 0xFF);
-}
-
void WebMediaPlayerImpl::paint(WebCanvas* canvas,
const WebRect& rect,
uint8_t alpha) {
@@ -625,22 +618,6 @@ COMPILE_ASSERT_MATCHING_STATUS_ENUM(AddIdStatusReachedIdLimit, kReachedIdLimit);
WebKit::WebMediaPlayer::AddIdStatus WebMediaPlayerImpl::sourceAddId(
const WebKit::WebString& id,
- const WebKit::WebString& type) {
- DCHECK_EQ(main_loop_, MessageLoop::current());
-
- WebKit::WebString kDefaultSourceType("video/webm; codecs=\"vp8, vorbis\"");
-
- if (type != kDefaultSourceType)
- return WebKit::WebMediaPlayer::AddIdStatusNotSupported;
-
- WebKit::WebVector<WebKit::WebString> codecs(static_cast<size_t>(2));
- codecs[0] = "vp8";
- codecs[1] = "vorbis";
- return sourceAddId(id, "video/webm", codecs);
-}
-
-WebKit::WebMediaPlayer::AddIdStatus WebMediaPlayerImpl::sourceAddId(
- const WebKit::WebString& id,
const WebKit::WebString& type,
const WebKit::WebVector<WebKit::WebString>& codecs) {
DCHECK_EQ(main_loop_, MessageLoop::current());
@@ -653,11 +630,6 @@ WebKit::WebMediaPlayer::AddIdStatus WebMediaPlayerImpl::sourceAddId(
new_codecs));
}
-bool WebMediaPlayerImpl::sourceTimestampOffset(
- const WebKit::WebString& id, double offset) {
- return proxy_->DemuxerSetTimestampOffset(id.utf8().data(), offset);
-}
-
bool WebMediaPlayerImpl::sourceRemoveId(const WebKit::WebString& id) {
DCHECK(!id.isEmpty());
proxy_->DemuxerRemoveId(id.utf8().data());
@@ -669,12 +641,6 @@ WebKit::WebTimeRanges WebMediaPlayerImpl::sourceBuffered(
return ConvertToWebTimeRanges(proxy_->DemuxerBufferedRange(id.utf8().data()));
}
-bool WebMediaPlayerImpl::sourceAppend(const unsigned char* data,
- unsigned length) {
- return sourceAppend(WebKit::WebString::fromUTF8("DefaultSourceId"),
- data, length);
-}
-
bool WebMediaPlayerImpl::sourceAppend(const WebKit::WebString& id,
const unsigned char* data,
unsigned length) {

Powered by Google App Engine
This is Rietveld 408576698