From 7d9e848edb23ec6dae5b3679f5d69e43cbc41afc Mon Sep 17 00:00:00 2001
From: Jeremy Wu <jeremy24wu@gmail.com>
Date: Tue, 6 Aug 2019 21:58:46 +1000
Subject: [PATCH] use shared commands

---
 Plugins/Wox.Plugin.Url/Main.cs         | 12 ++----------
 Wox.Plugin/SharedCommands/SearchWeb.cs |  6 +++---
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git a/Plugins/Wox.Plugin.Url/Main.cs b/Plugins/Wox.Plugin.Url/Main.cs
index c3229064e1..1c3a708c84 100644
--- a/Plugins/Wox.Plugin.Url/Main.cs
+++ b/Plugins/Wox.Plugin.Url/Main.cs
@@ -1,10 +1,9 @@
 using System;
 using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
 using System.Text.RegularExpressions;
 using System.Windows.Controls;
 using Wox.Infrastructure.Storage;
+using Wox.Plugin.SharedCommands;
 
 namespace Wox.Plugin.Url
 {
@@ -80,15 +79,8 @@ namespace Wox.Plugin.Url
                             }
                             try
                             {
-                                var browserExecutableName = _settings.BrowserPath?.Split(new[] { System.IO.Path.DirectorySeparatorChar }, StringSplitOptions.None).Last();
-
-                                var browser = string.IsNullOrEmpty(browserExecutableName) ? "chrome" : _settings.BrowserPath;
+                                raw.NewBrowserWindow(_settings.BrowserPath);
                                 
-                                // Internet Explorer will open url in new browser window, and does not take the --new-window parameter
-                                var browserArguements = browserExecutableName == "iexplore.exe" ? raw : "--new-window " + raw;
-
-                                Process.Start(browser, browserArguements);                                                               
-
                                 return true;
                             }
                             catch(Exception ex)
diff --git a/Wox.Plugin/SharedCommands/SearchWeb.cs b/Wox.Plugin/SharedCommands/SearchWeb.cs
index 7c2cc31813..0dc03ee6d5 100644
--- a/Wox.Plugin/SharedCommands/SearchWeb.cs
+++ b/Wox.Plugin/SharedCommands/SearchWeb.cs
@@ -3,14 +3,14 @@ using System.Diagnostics;
 using System.IO;
 using System.Linq;
 
-namespace Wox.Plugin.WebSearch.Commands
+namespace Wox.Plugin.SharedCommands
 {
-    internal static class SearchWeb
+    public static class SearchWeb
     {
         /// <summary> Opens search in a new browser. If no browser path is passed in then Chrome is used. 
         /// Leave browser path blank to use Chrome.
         /// </summary>
-        internal static void NewBrowserWindow(this string url, string browserPath)
+        public static void NewBrowserWindow(this string url, string browserPath)
         {
             var browserExecutableName = browserPath?
                                         .Split(new[] { Path.DirectorySeparatorChar }, StringSplitOptions.None)