PDF accessibility tweaks

pdf/out_of_process_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_OUT_OF_PROCESS_INSTANCE_H_
 #define PDF_OUT_OF_PROCESS_INSTANCE_H_
 
 #include <stdint.h>
 #include <string.h>
 
@@ skipping 328 matching lines @@
   // zooming the plugin so that flickering doesn't occur while zooming.
   bool stop_scrolling_;
 
   // The background color of the PDF viewer.
   uint32_t background_color_;
 
   // The blank space above the first page of the document reserved for the
   // 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);
 };
 
 }  // namespace chrome_pdf
 
 #endif  // PDF_OUT_OF_PROCESS_INSTANCE_H_