cleanup safari specific features

src/background/commands.background.ts

@@ -52,7 +52,7 @@ export default class CommandsBackground {
     }
 
     private async generatePasswordToClipboard() {
-        if (this.isSafari || this.isEdge) {
+        if (this.isEdge) {
             // Safari does not support access to clipboard from background
             return;
         }

src/background/main.background.ts

@@ -368,6 +368,10 @@ export default class MainBackground {
         // Chrome APIs cannot open popup
 
         // TODO: Do we need to open this popup?
+        if (!this.isSafari) {
+            return;
+        }
+        await SafariApp.sendMessageToApp('showPopover');
 
         /*
         if (!this.isSafari || !safari.extension.toolbarItems || !safari.extension.toolbarItems.length) {

src/browser/browserApi.ts

@@ -230,7 +230,7 @@ export class BrowserApi {
         } else if (BrowserApi.isWebExtensionsApi || BrowserApi.isChromeApi) {
             win.close();
         } else if (BrowserApi.isSafariApi) {
-            SafariApp.sendMessageToApp('hideWindow');
+            SafariApp.sendMessageToApp('hidePopover');
         }
     }
 
@@ -293,7 +293,7 @@ export class BrowserApi {
         if (win != null) {
             return win.location.reload(true);
         } else if (BrowserApi.isSafariApi) {
-            SafariApp.sendMessageToApp('reloadWindow');
+            SafariApp.sendMessageToApp('reloadExtension');
         } else if (!BrowserApi.isSafariApi) {
             return chrome.runtime.reload();
         }

src/popup/settings/options.component.ts

@@ -68,7 +68,7 @@ export class OptionsComponent implements OnInit {
     }
 
     async ngOnInit() {
-        this.showDisableContextMenu = this.showClearClipboard = !this.platformUtilsService.isSafari();
+        this.showDisableContextMenu = !this.platformUtilsService.isSafari();
 
         this.enableAutoFillOnPageLoad = await this.storageService.get<boolean>(
             ConstantsService.enableAutoFillOnPageLoadKey);

src/popup/settings/settings.component.ts

@@ -94,7 +94,7 @@ export class SettingsComponent implements OnInit {
     }
 
     async saveLockOption(newValue: number) {
-        if (newValue == null && !this.platformUtilsService.isSafari()) {
+        if (newValue == null) {
             const confirmed = await this.platformUtilsService.showDialog(
                 this.i18nService.t('neverLockWarning'), null,
                 this.i18nService.t('yes'), this.i18nService.t('cancel'), 'warning');

src/safari/app/desktop/safari/SafariExtensionViewController.swift

@@ -135,9 +135,15 @@ class SafariExtensionViewController: SFSafariExtensionViewController, WKScriptMe
                             }
                         }
                     }
-                } else if command == "hideWindow" {
+                } else if command == "hidePopover" {
                     dismissPopover()
                     replyMessage(message: m!)
+                } else if command == "showPopover" {
+                    // TODO
+                    replyMessage(message: m!)
+                } else if command == "reloadExtension" {
+                    // TODO
+                    replyMessage(message: m!)
                 } else if command == "copyToClipboard" {
                     let pasteboard = NSPasteboard.general
                     pasteboard.declareTypes([NSPasteboard.PasteboardType.string], owner: nil)

src/services/browserPlatformUtils.service.ts

@@ -95,6 +95,7 @@ export default class BrowserPlatformUtilsService implements PlatformUtilsService
         }
 
         if (this.isSafari()) {
+            // TODO
             return false;
         }