Fixed the Pepper print interface to take in the print dimensions and printer DPI only in the PrintBegin method. The plugin is expected to remember this until PrintEnd is called.

BUG=None
TEST=Test printing with plugins that support the new interface

Review URL: http://codereview.chromium.org/858007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41453 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 11 deletions

diff --git a/third_party/npapi/bindings/npapi_extensions.h b/third_party/npapi/bindings/npapi_extensions.h
index 4c22ff0..2cfc717 100644
--- a/third_party/npapi/bindings/npapi_extensions.h
+++ b/third_party/npapi/bindings/npapi_extensions.h
@@ -492,30 +492,25 @@ struct _NPDeviceContextAudio {
 
 /* Being a print operation. Returns the total number of pages to print at the
  * given printableArea size and DPI. printableArea is in points (a point is 1/72
- * of an inch). */
+ * of an inch). The plugin is expected to remember the values of printableArea
+ * and printerDPI for use in subsequent print interface calls. These values
+ * should be cleared in printEnd*/
 typedef NPError (*NPPPrintBeginPtr) (
     NPP instance,
     NPRect* printableArea,
     int32 printerDPI,
     int32* numPages);
-/* Returns the required raster dimensions for the given printableArea
- * size and DPI. printableArea is in points (a point is 1/72 of an inch). */
+/* Returns the required raster dimensions for the given page. */
 typedef NPError (*NPPGetRasterDimensionsPtr) (
     NPP instance,
-    NPRect* printableArea,
-    int32 printerDPI,
+    int32 pageNumber,
     int32* widthInPixels,
     int32* heightInPixels);
-/* Prints the specified page on the given printableArea size and DPI.
- * printableArea is in points (a point is 1/72 of an inch). This allows the
- * plugin to print a raster output*/
+/* Prints the specified page This allows the plugin to print a raster output. */
 typedef NPError (*NPPPrintPageRasterPtr) (
     NPP instance,
     int32 pageNumber,
-    NPRect* printableArea,
-    int32 printerDPI,
     NPDeviceContext2D* printSurface);
-
 /* Ends the print operation */
 typedef NPError (*NPPPrintEndPtr) (NPP instance);