Improved Pinch-Zoom for PDF.

pdf/out_of_process_instance.h

Index: pdf/out_of_process_instance.h
diff --git a/pdf/out_of_process_instance.h b/pdf/out_of_process_instance.h
index a998537dfd1e27548cf580d50715c68c928afb51..364223d4d5df6cadc8c1ce230b22953953f7d7c5 100644
--- a/pdf/out_of_process_instance.h
+++ b/pdf/out_of_process_instance.h
@@ -235,9 +235,25 @@ class OutOfProcessInstance : public pp::Instance,
   // there are 10 pages, the height will be 8000).
   pp::Size document_size_;
 
-  double zoom_;  // Current zoom factor.
+  enum PinchPhase {

[Lei Zhang, 2016/10/27 16:56:35]

Add note that this match the .js version.

[Kevin McNee - google account, 2016/10/27 21:40:30]

Done.

+      PINCH_NONE = 0,
+      PINCH_START = 1,
+      PINCH_UPDATE_ZOOM_OUT = 2,
+      PINCH_UPDATE_ZOOM_IN = 3,
+      PINCH_END = 4
+  };
 
-  float device_scale_;  // Current device scale factor.
+  // Current zoom factor.
+  double zoom_;

[Lei Zhang, 2016/10/27 16:56:35]

Are we duplicating variables? e.g. pdf/pdfium/pdfium_engine.h already has a
|zoom_| variable, and a |position_| for the scroll position.

[Kevin McNee - google account, 2016/10/27 21:40:30]

This variable was here before this CL and it appears to be used in various
calculations in this class. Also, it looks like the zoom that pdfium_engine
stores is a combination of this zoom factor and the device scale (see
https://cs.chromium.org/chromium/src/pdf/out_of_process_instance.cc?l=1421).

+  double old_zoom_, initial_zoom_delta_;

[Lei Zhang, 2016/10/27 16:56:36]

1 decl per line please

[Kevin McNee - google account, 2016/10/27 21:40:30]

Done.

+  // True if we request a new bitmap rendering.
+  bool needs_reraster_;
+  // Scroll at the begining of zooming.

[Lei Zhang, 2016/10/27 16:56:36]

typo: beginning

[Kevin McNee - google account, 2016/10/27 21:40:30]

Done.

+  pp::FloatPoint last_current_scroll_;
+  // True if last bitmap was smaller than screen.
+  bool was_smaller_;
+  // Current device scale factor.
+  float device_scale_;
   // True if the plugin is full-page.
   bool full_;