Avoid OOB memcpy in chrome_pdf::CopyImage.

ppapi/cpp/image_data.h

Index: ppapi/cpp/image_data.h
diff --git a/ppapi/cpp/image_data.h b/ppapi/cpp/image_data.h
index e9366ceb41529cc23376762ce27ada3480a4d717..00bd0fd30738345f61f7c3233b9c882b7b8cb9e1 100644
--- a/ppapi/cpp/image_data.h
+++ b/ppapi/cpp/image_data.h
@@ -7,8 +7,9 @@
 
 #include "ppapi/c/ppb_image_data.h"
 #include "ppapi/cpp/point.h"
-#include "ppapi/cpp/size.h"
+#include "ppapi/cpp/rect.h"
 #include "ppapi/cpp/resource.h"
+#include "ppapi/cpp/size.h"
 
 /// @file
 /// This file defines the APIs for determining how a browser
@@ -129,6 +130,14 @@ class ImageData : public Resource {
   /// @return The address for the pixel.
   uint32_t* GetAddr32(const Point& coord);
 
+  /// This function determines if a rectangle's dimensions are fully contained
+  /// within the boundaries of the image.
+  ///
+  /// @param[in] rect A <code>Rect</code> representing the rectangle to check.
+  ///
+  /// @return True if fully contained, false otherwise.
+  bool Contains(const pp::Rect& rect) const;

[raymes, 2014/09/01 00:42:17]

Technically this is a change to the public API. We haven't needed this in the
long history of pp::ImageData, do you think we will start to need it a lot more?
If not, I think it might be better just to move a helper function into the place
where it's used. 

What do you think? :)

+
  private:
   void InitData();