WebKit merge 41447:41498

chrome/renderer/webmediaplayer_delegate_impl.cc

 // Copyright (c) 2008 The Chromium Authors. All rights reserved.  Use of this
 // source code is governed by a BSD-style license that can be found in the
 // LICENSE file.
 
 #include "chrome/renderer/media/audio_renderer_impl.h"
 #include "chrome/renderer/media/data_source_impl.h"
 #include "chrome/renderer/media/video_renderer_impl.h"
 #include "chrome/renderer/render_view.h"
 #include "chrome/renderer/webmediaplayer_delegate_impl.h"
 #include "googleurl/src/gurl.h"
@@ skipping 237 matching lines @@
 
   return pipeline_.GetBufferedBytes();
 }
 
 int64 WebMediaPlayerDelegateImpl::GetTotalBytes() const {
   DCHECK(main_loop_ && MessageLoop::current() == main_loop_);
 
   return pipeline_.GetTotalBytes();
 }
 
-void WebMediaPlayerDelegateImpl::SetRect(const gfx::Rect& rect) {
+void WebMediaPlayerDelegateImpl::SetSize(const gfx::Size& size) {
   DCHECK(main_loop_ && MessageLoop::current() == main_loop_);
 
   if (video_renderer_) {
-    video_renderer_->SetRect(rect);
+    // TODO(eroman): What to do here?
+    video_renderer_->SetRect(gfx::Rect(0, 0, size.width(), size.height()));

[scherkus (not reviewing), 2009/03/07 01:39:58]

Technically, we have to go in and change our stuff to use SetSize as well, but
this code will still work the same way.

You can put a TODO(scherkus) and we'll fix it later.

   }
 }
 
 void WebMediaPlayerDelegateImpl::Paint(skia::PlatformCanvas *canvas,
                                        const gfx::Rect& rect) {
   if (video_renderer_) {
     video_renderer_->Paint(canvas, rect);
   }
 }
 
@@ skipping 59 matching lines @@
     CancelableTask* task = new NotifyWebMediaPlayerTask(web_media_player_,
                                                         method);
     tasks_[index] = task;
     main_loop_->PostTask(FROM_HERE, task);
   }
 }
 
 void WebMediaPlayerDelegateImpl::PostRepaintTask() {
   PostTask(kRepaintTaskIndex, &webkit_glue::WebMediaPlayer::Repaint);
 }