Refactoring of the client-side input pipeline and scaling dimension management.

remoting/client/plugin/pepper_view_proxy.h

 // Copyright (c) 2011 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.
 
 // PepperViewProxy is used to invoke PepperView object on pepper thread. It
 // has the same interface as PepperView. When a method calls is received on
 // any chromoting threads it delegates the method call to pepper thread.
 // It also provide a detach mechanism so that when PepperView object is
 // destroyed PepperViewProxy will not call it anymore. This is important in
 // providing a safe shutdown of ChromotingInstance and PepperView.
 
 // This object is accessed on chromoting threads and pepper thread. The internal
 // PepperView object is only accessed on pepper thread so as the Detach() method
 // call.
 
 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_PROXY_H_
 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_PROXY_H_
 
 #include "base/memory/ref_counted.h"
 #include "remoting/client/plugin/pepper_view.h"
 
 namespace base {
 class MessageLoopProxy;
 }  // namespace base
 
 namespace remoting {
 
 class ChromotingInstance;
 
 class PepperViewProxy : public base::RefCountedThreadSafe<PepperViewProxy>,

[Sergey Ulanov, 2011/12/20 00:05:35]

FYI: We don't need this class anymore - network and UI work on the main plugin
thread anyway.

[Wez, 2011/12/20 07:14:14]

OK, gone!

                         public ChromotingView,
                         public FrameConsumer {
  public:
   PepperViewProxy(ChromotingInstance* instance, PepperView* view,
                   base::MessageLoopProxy* plugin_message_loop);
   virtual ~PepperViewProxy();
 
   // ChromotingView implementation.
   virtual bool Initialize() OVERRIDE;
   virtual void TearDown() OVERRIDE;
   virtual void Paint() OVERRIDE;
   virtual void SetSolidFill(uint32 color) OVERRIDE;
   virtual void UnsetSolidFill() OVERRIDE;
   virtual void SetConnectionState(
       protocol::ConnectionToHost::State state,
       protocol::ConnectionToHost::Error error) OVERRIDE;
 
-  // This method returns a value, so must run synchronously, so must be
-  // called only on the pepper thread.
-  virtual double GetHorizontalScaleRatio() const OVERRIDE;
-  virtual double GetVerticalScaleRatio() const OVERRIDE;
-
   // FrameConsumer implementation.
   virtual void AllocateFrame(media::VideoFrame::Format format,
                              const SkISize& size,
                              scoped_refptr<media::VideoFrame>* frame_out,
                              const base::Closure& done) OVERRIDE;
   virtual void ReleaseFrame(media::VideoFrame* frame) OVERRIDE;
   virtual void OnPartialFrameOutput(media::VideoFrame* frame,
                                     RectVector* rects,
                                     const base::Closure& done) OVERRIDE;
 
@@ skipping 15 matching lines @@
   PepperView* view_;
 
   scoped_refptr<base::MessageLoopProxy> plugin_message_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperViewProxy);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_PROXY_H_