Fix screen capture slowness in Chrome OS feedback report.

content/browser/renderer_host/media/desktop_capture_device_ash.cc

 // Copyright 2013 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 "content/browser/renderer_host/media/desktop_capture_device_ash.h"
 
 #include "base/logging.h"
 #include "base/timer/timer.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
@@ skipping 23 matching lines @@
  public:
   DesktopVideoCaptureMachine(const DesktopMediaID& source);
   virtual ~DesktopVideoCaptureMachine();
 
   // VideoCaptureFrameSource overrides.
   virtual bool Start(
       const scoped_refptr<ThreadSafeCaptureOracle>& oracle_proxy) OVERRIDE;
   virtual void Stop() OVERRIDE;
 
   // Implements aura::WindowObserver.
+  virtual void OnWindowBoundsChanged(aura::Window* window,
+                                     const gfx::Rect& old_bounds,
+                                     const gfx::Rect& new_bounds) OVERRIDE;
   virtual void OnWindowDestroyed(aura::Window* window) OVERRIDE;
 
   // Implements ui::CompositorObserver.
   virtual void OnCompositingDidCommit(ui::Compositor* compositor) OVERRIDE {}
   virtual void OnCompositingStarted(ui::Compositor* compositor,
                                     base::TimeTicks start_time) OVERRIDE {}
   virtual void OnCompositingEnded(ui::Compositor* compositor) OVERRIDE;
   virtual void OnCompositingAborted(ui::Compositor* compositor) OVERRIDE {}
   virtual void OnCompositingLockStateChanged(
       ui::Compositor* compositor) OVERRIDE {}
   virtual void OnUpdateVSyncParameters(ui::Compositor* compositor,
                                        base::TimeTicks timebase,
                                        base::TimeDelta interval) OVERRIDE {}
 
  private:
   // Captures a frame.
   // |dirty| is false for timer polls and true for compositor updates.
   void Capture(bool dirty);
 
+  // Update capture size. Must be called on the UI thread.
+  void UpdateCaptureSize();
+
   // Response callback for cc::Layer::RequestCopyOfOutput().
   void DidCopyOutput(
       scoped_refptr<media::VideoFrame> video_frame,
       base::Time start_time,
       const ThreadSafeCaptureOracle::CaptureFrameCallback& capture_frame_cb,
       scoped_ptr<cc::CopyOutputResult> result);
 
   // The window associated with the desktop.
   aura::Window* desktop_window_;
 
@@ skipping 37 matching lines @@
     return false;
 
   // If the desktop layer is already destroyed then return failure.
   desktop_layer_ = desktop_window_->layer();
   if (!desktop_layer_)
     return false;
 
   DCHECK(oracle_proxy.get());
   oracle_proxy_ = oracle_proxy;
 
+  // Update capture size.
+  UpdateCaptureSize();
+
   // Start observing window events.
   desktop_window_->AddObserver(this);
 
   // Start observing compositor updates.
   ui::Compositor* compositor = desktop_layer_->GetCompositor();
   if (!compositor)
     return false;
 
   compositor->AddObserver(this);
 
@@ skipping 19 matching lines @@
     if (compositor)
       compositor->RemoveObserver(this);
   }
 
   // Stop timer.
   timer_.Stop();
 
   started_ = false;
 }
 
+void DesktopVideoCaptureMachine::UpdateCaptureSize() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  if (oracle_proxy_ && desktop_layer_) {
+    oracle_proxy_->UpdateCaptureSize(ui::ConvertSizeToPixel(
+        desktop_layer_, desktop_layer_->bounds().size()));
+  }
+}
+
 void DesktopVideoCaptureMachine::Capture(bool dirty) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   // Do not capture if the desktop layer is already destroyed.
   if (!desktop_layer_)
     return;
 
   scoped_refptr<media::VideoFrame> frame;
   ThreadSafeCaptureOracle::CaptureFrameCallback capture_frame_cb;
 
@@ skipping 72 matching lines @@
                                              region_in_frame,
                                              true,
                                              true));
   }
   yuv_readback_pipeline_->ReadbackYUV(
       texture_mailbox.name(), texture_mailbox.sync_point(), video_frame.get(),
       base::Bind(&CopyOutputFinishedForVideo, start_time, capture_frame_cb,
                  base::Passed(&release_callback)));
 }
 
+void DesktopVideoCaptureMachine::OnWindowBoundsChanged(
+    aura::Window* window,
+    const gfx::Rect& old_bounds,
+    const gfx::Rect& new_bounds) {
+  DCHECK(desktop_window_ && window == desktop_window_);
+
+  // Post task to update capture size on UI thread.
+  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(
+      &DesktopVideoCaptureMachine::UpdateCaptureSize, AsWeakPtr()));
+}
+
 void DesktopVideoCaptureMachine::OnWindowDestroyed(aura::Window* window) {
   DCHECK(desktop_window_ && window == desktop_window_);
   desktop_window_ = NULL;
   desktop_layer_ = NULL;
 
   // Post task to stop capture on UI thread.
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(
       &DesktopVideoCaptureMachine::Stop, AsWeakPtr()));
 }
 
@@ skipping 29 matching lines @@
     scoped_ptr<Client> client) {
   DVLOG(1) << "Allocating " << params.requested_format.frame_size.ToString();
   impl_->AllocateAndStart(params, client.Pass());
 }
 
 void DesktopCaptureDeviceAsh::StopAndDeAllocate() {
   impl_->StopAndDeAllocate();
 }
 
 }  // namespace content