Fix Toolbar requiring SearchView.init having already been called

Author: cdrini

src/plugins/search/plugin.search.js

@@ -66,23 +66,28 @@ BookReader.prototype.setup = (function (super_) {
     /** @type { {[pageIndex: number]: SearchInsideMatchBox[]} } */
     this._searchBoxesByIndex = {};
 
-    this.searchView = undefined;
+    if (this.enableSearch) {
+      this.searchView = new SearchView({
+        br: this,
+        searchCancelledCallback: () => {
+          this._cancelSearch();
+          this.trigger('SearchCanceled', { term: this.searchTerm, instance: this });
+        }
+      });
+    } else {
+      this.searchView = null;
+    }
   };
 })(BookReader.prototype.setup);
 
 /** @override */
 BookReader.prototype.init = (function (super_) {
   return function () {
     super_.call(this);
-    // give SearchView the most complete bookreader state
-    this.searchView = new SearchView({
-      br: this,
-      searchCancelledCallback: () => {
-        this._cancelSearch();
-        this.trigger('SearchCanceled', { term: this.searchTerm, instance: this });
-      }
-    });
-    if (this.options.enableSearch && this.options.initialSearchTerm) {
+    if (!this.enableSearch) return;
+
+    this.searchView.init();
+    if (this.options.initialSearchTerm) {
       /**
        * this.search() take two parameter
        * 1. this.options.initialSearchTerm - search term

src/plugins/search/view.js

@@ -11,10 +11,13 @@ class SearchView {
     this.br = br;
     this.matches = [];
     this.cacheDOMElements();
-    this.bindEvents();
     this.cancelSearch = searchCancelledCallback;
   }
 
+  init() {
+    this.bindEvents();
+  }
+
   cacheDOMElements() {
     this.dom = {};
     // Search input within the top toolbar. Will be removed once the mobile menu is replaced.