[Chrome] Support NumCopies print preset

pdf/instance.cc

Index: pdf/instance.cc
diff --git a/pdf/instance.cc b/pdf/instance.cc
index 6a68400d6ee4e185291c7c5c8fcf7d9174056752..66aff00aef108f2ab2a08b83eefd4da9ed80193d 100644
--- a/pdf/instance.cc
+++ b/pdf/instance.cc
@@ -188,7 +188,7 @@ const ToolbarButtonInfo kPrintPreviewToolbarButtons[] = {
       PP_RESOURCEIMAGE_PDF_BUTTON_ZOOMIN_END_PRESSED },
 };
 
-static const char kPPPPdfInterface[] = PPP_PDF_INTERFACE_1;
+static const char kPPPPdfInterface[] = PPP_PDF_INTERFACE_2;
 
 PP_Var GetLinkAtPosition(PP_Instance instance, PP_Point point) {
   pp::Var var;
@@ -215,9 +215,20 @@ void Transform(PP_Instance instance, PP_PrivatePageTransformType type) {
   }
 }
 
+PP_Bool GetPrintPresetOptionsFromDocument(PP_Instance instance,
+                                          PP_PrintPresetOptions_Dev& options) {
+  void* object = pp::Instance::GetPerInstanceObject(instance, kPPPPdfInterface);
+  PP_Bool ret = PP_FALSE;
+  if (object)
+    ret = static_cast<Instance*>(object)
+              ->GetPrintPresetOptionsFromDocument(options);

[dmichael (off chromium), 2014/09/22 18:11:51]

The style here looks funny. I think normally we'd have -> on the previous line,
and the 4-space indent would probably be count from where "ret" starts.

How about just having a "Instance* obj_instance" stack variable, like Transform
does?

[Nikhil, 2014/09/23 10:32:28]

The style here was a result of "git cl format". I've modified the code to make
use of stack variable.

+  return ret;
+}
+
 const PPP_Pdf ppp_private = {
   &GetLinkAtPosition,
-  &Transform
+  &Transform,
+  &GetPrintPresetOptionsFromDocument
 };
 
 int ExtractPrintPreviewPageIndex(const std::string& src_url) {
@@ -652,6 +663,14 @@ pp::Var Instance::GetInstanceObject() {
   return instance_object_;
 }
 
+PP_Bool Instance::GetPrintPresetOptionsFromDocument(
+    PP_PrintPresetOptions_Dev& options) {
+  options.is_scaling_disabled = PP_FromBool(IsPrintScalingDisabled());
+  options.copies = engine_->GetCopiesToPrint();
+
+  return PP_TRUE;
+}
+
 pp::Var Instance::GetLinkAtPosition(const pp::Point& point) {
   pp::Point offset_point(point);
   ScalePoint(device_scale_, &offset_point);