Prevent the in-process PDF plugin re-entering into JS during blink layout

pdf/instance.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef PDF_INSTANCE_H_
 #define PDF_INSTANCE_H_
 
 #include <queue>
 #include <set>
 #include <string>
@@ skipping 184 matching lines @@
 
   // Helper functions for implementing PPP_PDF.
   void RotateClockwise();
   void RotateCounterclockwise();
 
  private:
   // Called whenever the plugin geometry changes to update the location of the
   // scrollbars, background parts, and notifies the pdf engine.
   void OnGeometryChanged(double old_zoom, float old_device_scale);
 
+  // Runs the given JS callback given in |callback|.
+  void RunCallback(int32_t, pp::Var callback);
+
   void CreateHorizontalScrollbar();
   void CreateVerticalScrollbar();
   void DestroyHorizontalScrollbar();
   void DestroyVerticalScrollbar();
 
   // Returns the thickness of a scrollbar. This returns the thickness when it's
   // shown, so for overlay scrollbars it'll still be non-zero.
   int GetScrollbarThickness();
 
   // Returns the space we need to reserve for the scrollbar in the plugin area.
@@ skipping 234 matching lines @@
   uint32 delayed_progress_timer_id_;
   PageIndicator page_indicator_;
 
   // Used for creating images from numbers.
   scoped_ptr<NumberImageGenerator> number_image_generator_;
 
   // Used for submitting forms.
   pp::CompletionCallbackFactory<Instance> form_factory_;
   pp::URLLoader form_loader_;
 
-  // Used for printing without re-entrancy issues.
-  pp::CompletionCallbackFactory<Instance> print_callback_factory_;
+  // Used for generating callbacks.
+  // TODO(raymes): We don't really need other callback factories we can just
+  // fold them into this one.
+  pp::CompletionCallbackFactory<Instance> callback_factory_;
 
   // True if we haven't painted the plugin viewport yet.
   bool first_paint_;
 
   // True when we've painted at least one page from the document.
   bool painted_first_page_;
 
   // True if we should display page indicator, false otherwise
   bool show_page_indicator_;
 
@@ skipping 46 matching lines @@
                        const std::vector<pp::Var>& args,
                        pp::Var* exception);
 
  private:
   Instance* instance_;
 };
 
 }  // namespace chrome_pdf
 
 #endif  // PDF_INSTANCE_H_