PDF accessibility tweaks

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 2b0ebee7daebe38aeda07ebfeb535837d5cac898..10f293196f26e3d6cabcb124162f3e0486278bb4 100644
--- a/pdf/out_of_process_instance.h
+++ b/pdf/out_of_process_instance.h
@@ -346,9 +346,14 @@ class OutOfProcessInstance : public pp::Instance,
   // toolbar.
   int top_toolbar_height_;
 
-  // Whether we should send accessibility information about the loaded PDF.
+  // Whether we should send accessibility information about the PDF when
+  // it loads.
   bool accessibility_enabled_;
 
+  // Whether we've already started sending accessibility information about
+  // the PDF.
+  bool accessibility_loaded_;

[Lei Zhang, 2016/06/09 22:19:05]

Are all 4 combinations of the two a11y variables reachable? Do you think an enum
might better description the state we are in?

[dmazzoni, 2016/06/10 17:44:52]

Good idea, take a look.

+
   DISALLOW_COPY_AND_ASSIGN(OutOfProcessInstance);
 };