Introduce new test for bookmark loading.

chrome/browser/resources/pdf/pdf.js

 // Copyright 2013 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.
 
 'use strict';
 
 /**
  * @return {number} Width of a scrollbar in pixels
  */
 function getScrollbarWidth() {
@@ skipping 27 matching lines @@
 
 /**
  * Creates a new PDFViewer. There should only be one of these objects per
  * document.
  * @param {Object} streamDetails The stream object which points to the data
  *     contained in the PDF.
  */
 function PDFViewer(streamDetails) {
   this.streamDetails = streamDetails;
   this.loaded = false;
+  this.bookmarksLoaded = false;
 
   // The sizer element is placed behind the plugin element to cause scrollbars
   // to be displayed in the window. It is sized according to the document size
   // of the pdf and zoom level.
   this.sizer_ = $('sizer');
   this.toolbar_ = $('toolbar');
   this.pageIndicator_ = $('page-indicator');
   this.progressBar_ = $('progress-bar');
   this.passwordScreen_ = $('password-screen');
   this.passwordScreen_.addEventListener('password-submitted',
@@ skipping 356 matching lines @@
         this.passwordScreen_.text = message.data.getPasswordString;
         this.progressBar_.text = message.data.loadingString;
         this.progressBar_.style.visibility = 'visible';
         this.errorScreen_.text = message.data.loadFailedString;
         break;
       case 'cancelStreamUrl':
         chrome.streamsPrivate.abort(this.streamDetails.streamUrl);
         break;
       case 'bookmarks':
         this.bookmarks = message.data.bookmarks;
-        console.log(this.bookmarks);
+        this.bookmarksLoaded = true;
+        var loadEvent = new Event('bookmarksload');
+        window.dispatchEvent(loadEvent);
+        this.sendScriptingMessage_({
+          type: 'bookmarksLoaded'
+        });

[raymes, 2015/01/06 05:52:29]

if we ensure that bookmark information is sent here before the document is
loaded, we wouldn't need to add a new event.

[raymes, 2015/01/13 07:25:12]

(we also wouldn't need the bookmarksLoaded member either :)

[Alexandre Carlton, 2015/01/15 05:01:41]

Done.

         break;
     }
   },
 
   /**
    * @private
    * A callback that's called before the zoom changes. Notify the plugin to stop
    * reacting to scroll events while zoom is taking place to avoid flickering.
    */
   beforeZoom_: function() {
@@ skipping 172 matching lines @@
               this.streamDetails.tabId != -1);
   },
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };