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

Unified Diff: trunk/src/remoting/client/plugin/pepper_view.h

Issue 24217003: Revert 224101 "Remove dependency on Skia from chromoting client." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 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: trunk/src/remoting/client/plugin/pepper_view.h
===================================================================
--- trunk/src/remoting/client/plugin/pepper_view.h (revision 224204)
+++ trunk/src/remoting/client/plugin/pepper_view.h (working copy)
@@ -16,8 +16,6 @@
#include "ppapi/cpp/point.h"
#include "ppapi/utility/completion_callback_factory.h"
#include "remoting/client/frame_consumer.h"
-#include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
-#include "third_party/webrtc/modules/desktop_capture/desktop_region.h"
namespace base {
class Time;
@@ -43,26 +41,26 @@
virtual ~PepperView();
// FrameConsumer implementation.
- virtual void ApplyBuffer(const webrtc::DesktopSize& view_size,
- const webrtc::DesktopRect& clip_area,
+ virtual void ApplyBuffer(const SkISize& view_size,
+ const SkIRect& clip_area,
webrtc::DesktopFrame* buffer,
- const webrtc::DesktopRegion& region) OVERRIDE;
+ const SkRegion& region) OVERRIDE;
virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) OVERRIDE;
- virtual void SetSourceSize(const webrtc::DesktopSize& source_size,
- const webrtc::DesktopVector& dpi) OVERRIDE;
+ virtual void SetSourceSize(const SkISize& source_size,
+ const SkIPoint& dpi) OVERRIDE;
// Updates the PepperView's size & clipping area, taking into account the
// DIP-to-device scale factor.
void SetView(const pp::View& view);
// Returns the dimensions of the most recently displayed frame, in pixels.
- const webrtc::DesktopSize& get_source_size() const {
+ const SkISize& get_source_size() const {
return source_size_;
}
// Return the dimensions of the view in Density Independent Pixels (DIPs).
// Note that there may be multiple device pixels per DIP.
- const webrtc::DesktopSize& get_view_size_dips() const {
+ const SkISize& get_view_size_dips() const {
return dips_size_;
}
@@ -82,9 +80,9 @@
// clip area of the view has changed since the buffer was generated then
// FrameProducer is supplied the missed parts of |region|. The FrameProducer
// will be supplied a new buffer when FlushBuffer() completes.
- void FlushBuffer(const webrtc::DesktopRect& clip_area,
+ void FlushBuffer(const SkIRect& clip_area,
webrtc::DesktopFrame* buffer,
- const webrtc::DesktopRegion& region);
+ const SkRegion& region);
// Handles completion of FlushBuffer(), triggering a new buffer to be
// returned to FrameProducer for rendering.
@@ -109,11 +107,11 @@
// Queued buffer to paint, with clip area and dirty region in device pixels.
webrtc::DesktopFrame* merge_buffer_;
- webrtc::DesktopRect merge_clip_area_;
- webrtc::DesktopRegion merge_region_;
+ SkIRect merge_clip_area_;
+ SkRegion merge_region_;
// View size in Density Independent Pixels (DIPs).
- webrtc::DesktopSize dips_size_;
+ SkISize dips_size_;
// Scale factor from DIPs to device pixels.
float dips_to_device_scale_;
@@ -121,19 +119,19 @@
// View size in output pixels. This is the size at which FrameProducer must
// render frames. It usually matches the DIPs size of the view, but may match
// the size in device pixels when scaling is in effect, to reduce artefacts.
- webrtc::DesktopSize view_size_;
+ SkISize view_size_;
// Scale factor from output pixels to device pixels.
float dips_to_view_scale_;
// Visible area of the view, in output pixels.
- webrtc::DesktopRect clip_area_;
+ SkIRect clip_area_;
// Size of the most recent source frame in pixels.
- webrtc::DesktopSize source_size_;
+ SkISize source_size_;
// Resolution of the most recent source frame dots-per-inch.
- webrtc::DesktopVector source_dpi_;
+ SkIPoint source_dpi_;
// True if there is already a Flush() pending on the Graphics2D context.
bool flush_pending_;
« no previous file with comments | « trunk/src/remoting/client/plugin/chromoting_instance.cc ('k') | trunk/src/remoting/client/plugin/pepper_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698