General refactoring (#2016)

* Remove // Public comments

* Fix errors

* Remove source parameter from Popup*.setOptionsContext

1 files changed, 4 insertions, 9 deletions

diff --git a/ext/js/app/popup-window.js b/ext/js/app/popup-window.js
index 61c627b8..8f1a8355 100644
--- a/ext/js/app/popup-window.js
+++ b/ext/js/app/popup-window.js
@@ -38,8 +38,6 @@ class PopupWindow extends EventDispatcher {
         this._popupTabId = null;
     }
 
-    // Public properties
-
     /**
      * The ID of the popup.
      * @type {string}
@@ -59,7 +57,7 @@ class PopupWindow extends EventDispatcher {
      * @type {Popup}
      */
     set parent(value) {
-        throw new Error('Not supported on PopupProxy');
+        throw new Error('Not supported on PopupWindow');
     }
 
     /**
@@ -77,7 +75,7 @@ class PopupWindow extends EventDispatcher {
      * @throws Throws an error, since this class doesn't support children.
      */
     set child(value) {
-        throw new Error('Not supported on PopupProxy');
+        throw new Error('Not supported on PopupWindow');
     }
 
     /**
@@ -113,16 +111,13 @@ class PopupWindow extends EventDispatcher {
         return this._frameId;
     }
 
-
-    // Public functions
-
     /**
      * Sets the options context for the popup.
      * @param {object} optionsContext The options context object.
      * @returns {Promise<void>}
      */
-    setOptionsContext(optionsContext, source) {
-        return this._invoke(false, 'setOptionsContext', {id: this._id, optionsContext, source});
+    setOptionsContext(optionsContext) {
+        return this._invoke(false, 'setOptionsContext', {id: this._id, optionsContext});
     }
 
     /**