add spaces in exception messages (#2620)

campersau · amaitland · commit 6e82717f3f43 · 2019-01-12T08:00:07.000+10:00
diff --git a/CefSharp.Core/Cef.h b/CefSharp.Core/Cef.h
@@ -421,10 +421,10 @@ namespace CefSharp
                 {
                     if (_initializedThreadId != Thread::CurrentThread->ManagedThreadId)
                     {
-                        throw gcnew Exception("Cef.Shutdown must be called on the same thread that Cef.Initialize was called - typically your UI thread." +
-                            "If you called Cef.Initialize on a Thread other than the UI thread then you will need to call Cef.Shutdown on the same thread." +
-                            "Cef.Initialize was called on ManagedThreadId: " + _initializedThreadId + "where Cef.Shutdown is being called on" +
-                            "ManagedThreadId:" + Thread::CurrentThread->ManagedThreadId);
+                        throw gcnew Exception("Cef.Shutdown must be called on the same thread that Cef.Initialize was called - typically your UI thread. " +
+                            "If you called Cef.Initialize on a Thread other than the UI thread then you will need to call Cef.Shutdown on the same thread. " +
+                            "Cef.Initialize was called on ManagedThreadId: " + _initializedThreadId + "where Cef.Shutdown is being called on " +
+                            "ManagedThreadId: " + Thread::CurrentThread->ManagedThreadId);
                     }
 
                     UIThreadTaskFactory = nullptr;
diff --git a/CefSharp.Example/Handlers/BrowserProcessHandler.cs b/CefSharp.Example/Handlers/BrowserProcessHandler.cs
@@ -37,7 +37,7 @@ void IBrowserProcessHandler.OnContextInitialized()
 
                         foreach (var cookie in cookies)
                         {
-                            Debug.WriteLine("CookieName:" + cookie.Name);
+                            Debug.WriteLine("CookieName: " + cookie.Name);
                         }
                     }
                     else
@@ -54,7 +54,7 @@ void IBrowserProcessHandler.OnContextInitialized()
 
                         foreach (var cookie in cookies)
                         {
-                            Debug.WriteLine("CookieName:" + cookie.Name);
+                            Debug.WriteLine("CookieName: " + cookie.Name);
                         }
                     }
                     else
diff --git a/CefSharp.OffScreen.Example/Program.cs b/CefSharp.OffScreen.Example/Program.cs
@@ -75,7 +75,7 @@ await Cef.UIThreadTaskFactory.StartNew(delegate
                     //Check do not track status
                     var doNotTrack = (bool)preferences["enable_do_not_track"];
 
-                    Debug.WriteLine("DoNotTrack:" + doNotTrack);
+                    Debug.WriteLine("DoNotTrack: " + doNotTrack);
                 });
 
                 var onUi = Cef.CurrentlyOnThread(CefThreadIds.TID_UI);
diff --git a/CefSharp.OffScreen/ChromiumWebBrowser.cs b/CefSharp.OffScreen/ChromiumWebBrowser.cs
@@ -549,7 +549,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op
 
             if (IsBrowserInitialized)
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
 
@@ -590,7 +590,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio
 
             if (IsBrowserInitialized)
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
 
diff --git a/CefSharp.WinForms.Example/Program.cs b/CefSharp.WinForms.Example/Program.cs
@@ -35,7 +35,7 @@ public static int Main(string[] args)
 #if DEBUG
                 if (!System.Diagnostics.Debugger.IsAttached)
                 {
-                    MessageBox.Show("When running this Example outside of Visual Studio" +
+                    MessageBox.Show("When running this Example outside of Visual Studio " +
                                     "please make sure you compile in `Release` mode.", "Warning");
                 }
 #endif
@@ -53,7 +53,7 @@ public static int Main(string[] args)
 #if DEBUG
                 if (!System.Diagnostics.Debugger.IsAttached)
                 {
-                    MessageBox.Show("When running this Example outside of Visual Studio" +
+                    MessageBox.Show("When running this Example outside of Visual Studio " +
                                     "please make sure you compile in `Release` mode.", "Warning");
                 }
 #endif
diff --git a/CefSharp.WinForms/ChromiumWebBrowser.cs b/CefSharp.WinForms/ChromiumWebBrowser.cs
@@ -79,7 +79,7 @@ public BrowserSettings BrowserSettings
             {
                 if (browserCreated)
                 {
-                    throw new Exception("Browser has already been created. BrowserSettings must be" +
+                    throw new Exception("Browser has already been created. BrowserSettings must be " +
                                         "set before the underlying CEF browser is created.");
                 }
                 browserSettings = value;
@@ -97,7 +97,7 @@ public IRequestContext RequestContext
             {
                 if (browserCreated)
                 {
-                    throw new Exception("Browser has already been created. RequestContext must be" +
+                    throw new Exception("Browser has already been created. RequestContext must be " +
                                         "set before the underlying CEF browser is created.");
                 }
                 if (value != null && value.GetType() != typeof(RequestContext))
@@ -539,7 +539,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op
 
             if (IsBrowserInitialized)
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
 
@@ -582,7 +582,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio
 
             if (IsBrowserInitialized)
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
 
diff --git a/CefSharp.Wpf/ChromiumWebBrowser.cs b/CefSharp.Wpf/ChromiumWebBrowser.cs
@@ -131,7 +131,7 @@ public BrowserSettings BrowserSettings
             {
                 if (browserCreated)
                 {
-                    throw new Exception("Browser has already been created. BrowserSettings must be" +
+                    throw new Exception("Browser has already been created. BrowserSettings must be " +
                                         "set before the underlying CEF browser is created.");
                 }
 
@@ -157,7 +157,7 @@ public IRequestContext RequestContext
             {
                 if (browserCreated)
                 {
-                    throw new Exception("Browser has already been created. RequestContext must be" +
+                    throw new Exception("Browser has already been created. RequestContext must be " +
                                         "set before the underlying CEF browser is created.");
                 }
                 if (value != null && value.GetType() != typeof(RequestContext))
@@ -2299,7 +2299,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op
 
             if (InternalIsBrowserInitialized())
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
 
@@ -2340,7 +2340,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio
 
             if (InternalIsBrowserInitialized())
             {
-                throw new Exception("Browser is already initialized. RegisterJsObject must be" +
+                throw new Exception("Browser is already initialized. RegisterJsObject must be " +
                                     "called before the underlying CEF browser is created.");
             }
             var objectRepository = managedCefBrowserAdapter.JavascriptObjectRepository;
diff --git a/CefSharp/WebBrowserExtensions.cs b/CefSharp/WebBrowserExtensions.cs
@@ -1043,10 +1043,10 @@ private static void ThrowExceptionIfBrowserHostNull(IBrowserHost browserHost)
 
         private static void ThrowExceptionIfCanExecuteJavascriptInMainFrameFalse()
         {
-            throw new Exception("Unable to execute javascript at this time, scripts can only be executed within a V8Context." +
-                                    "Use the IWebBrowser.CanExecuteJavascriptInMainFrame property to guard against this exception." +
+            throw new Exception("Unable to execute javascript at this time, scripts can only be executed within a V8Context. " +
+                                    "Use the IWebBrowser.CanExecuteJavascriptInMainFrame property to guard against this exception. " +
                                     "See https://github.com/cefsharp/CefSharp/wiki/General-Usage#when-can-i-start-executing-javascript " +
-                                    "for more details on when you can execute javascript. For frames that do not contain Javascript then no" +
+                                    "for more details on when you can execute javascript. For frames that do not contain Javascript then no " +
                                     "V8Context will be created. Executing a script once the frame has loaded it's possible to create a V8Context. " +
                                     "You can use browser.GetMainFrame().ExecuteJavaScriptAsync(script) or browser.GetMainFrame().EvaluateScriptAsync " +
                                     "to bypass these checks (advanced users only).");

Original file line number	Diff line number	Diff line change
`@@ -421,10 +421,10 @@ namespace CefSharp`
`421`	`421`	`{`
`422`	`422`	`if (_initializedThreadId != Thread::CurrentThread->ManagedThreadId)`
`423`	`423`	`{`
`424`		`- throw gcnew Exception("Cef.Shutdown must be called on the same thread that Cef.Initialize was called - typically your UI thread." +`
`425`		`- "If you called Cef.Initialize on a Thread other than the UI thread then you will need to call Cef.Shutdown on the same thread." +`
`426`		`- "Cef.Initialize was called on ManagedThreadId: " + _initializedThreadId + "where Cef.Shutdown is being called on" +`
`427`		`- "ManagedThreadId:" + Thread::CurrentThread->ManagedThreadId);`
	`424`	`+ throw gcnew Exception("Cef.Shutdown must be called on the same thread that Cef.Initialize was called - typically your UI thread. " +`
	`425`	`+ "If you called Cef.Initialize on a Thread other than the UI thread then you will need to call Cef.Shutdown on the same thread. " +`
	`426`	`+ "Cef.Initialize was called on ManagedThreadId: " + _initializedThreadId + "where Cef.Shutdown is being called on " +`
	`427`	`+ "ManagedThreadId: " + Thread::CurrentThread->ManagedThreadId);`
`428`	`428`	`}`
`429`	`429`
`430`	`430`	`UIThreadTaskFactory = nullptr;`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ void IBrowserProcessHandler.OnContextInitialized()`
`37`	`37`
`38`	`38`	`foreach (var cookie in cookies)`
`39`	`39`	`{`
`40`		`- Debug.WriteLine("CookieName:" + cookie.Name);`
	`40`	`+ Debug.WriteLine("CookieName: " + cookie.Name);`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`	`else`
`@@ -54,7 +54,7 @@ void IBrowserProcessHandler.OnContextInitialized()`
`54`	`54`
`55`	`55`	`foreach (var cookie in cookies)`
`56`	`56`	`{`
`57`		`- Debug.WriteLine("CookieName:" + cookie.Name);`
	`57`	`+ Debug.WriteLine("CookieName: " + cookie.Name);`
`58`	`58`	`}`
`59`	`59`	`}`
`60`	`60`	`else`
Original file line number	Diff line number	Diff line change
`@@ -549,7 +549,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op`
`549`	`549`
`550`	`550`	`if (IsBrowserInitialized)`
`551`	`551`	`{`
`552`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`552`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`553`	`553`	`"called before the underlying CEF browser is created.");`
`554`	`554`	`}`
`555`	`555`
`@@ -590,7 +590,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio`
`590`	`590`
`591`	`591`	`if (IsBrowserInitialized)`
`592`	`592`	`{`
`593`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`593`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`594`	`594`	`"called before the underlying CEF browser is created.");`
`595`	`595`	`}`
`596`	`596`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public static int Main(string[] args)`
`35`	`35`	`#if DEBUG`
`36`	`36`	`if (!System.Diagnostics.Debugger.IsAttached)`
`37`	`37`	`{`
`38`		`- MessageBox.Show("When running this Example outside of Visual Studio" +`
	`38`	`+ MessageBox.Show("When running this Example outside of Visual Studio " +`
`39`	`39`	"please make sure you compile in `Release` mode.", "Warning");
`40`	`40`	`}`
`41`	`41`	`#endif`
`@@ -53,7 +53,7 @@ public static int Main(string[] args)`
`53`	`53`	`#if DEBUG`
`54`	`54`	`if (!System.Diagnostics.Debugger.IsAttached)`
`55`	`55`	`{`
`56`		`- MessageBox.Show("When running this Example outside of Visual Studio" +`
	`56`	`+ MessageBox.Show("When running this Example outside of Visual Studio " +`
`57`	`57`	"please make sure you compile in `Release` mode.", "Warning");
`58`	`58`	`}`
`59`	`59`	`#endif`
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public BrowserSettings BrowserSettings`
`79`	`79`	`{`
`80`	`80`	`if (browserCreated)`
`81`	`81`	`{`
`82`		`- throw new Exception("Browser has already been created. BrowserSettings must be" +`
	`82`	`+ throw new Exception("Browser has already been created. BrowserSettings must be " +`
`83`	`83`	`"set before the underlying CEF browser is created.");`
`84`	`84`	`}`
`85`	`85`	`browserSettings = value;`
`@@ -97,7 +97,7 @@ public IRequestContext RequestContext`
`97`	`97`	`{`
`98`	`98`	`if (browserCreated)`
`99`	`99`	`{`
`100`		`- throw new Exception("Browser has already been created. RequestContext must be" +`
	`100`	`+ throw new Exception("Browser has already been created. RequestContext must be " +`
`101`	`101`	`"set before the underlying CEF browser is created.");`
`102`	`102`	`}`
`103`	`103`	`if (value != null && value.GetType() != typeof(RequestContext))`
`@@ -539,7 +539,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op`
`539`	`539`
`540`	`540`	`if (IsBrowserInitialized)`
`541`	`541`	`{`
`542`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`542`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`543`	`543`	`"called before the underlying CEF browser is created.");`
`544`	`544`	`}`
`545`	`545`
`@@ -582,7 +582,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio`
`582`	`582`
`583`	`583`	`if (IsBrowserInitialized)`
`584`	`584`	`{`
`585`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`585`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`586`	`586`	`"called before the underlying CEF browser is created.");`
`587`	`587`	`}`
`588`	`588`
Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public BrowserSettings BrowserSettings`
`131`	`131`	`{`
`132`	`132`	`if (browserCreated)`
`133`	`133`	`{`
`134`		`- throw new Exception("Browser has already been created. BrowserSettings must be" +`
	`134`	`+ throw new Exception("Browser has already been created. BrowserSettings must be " +`
`135`	`135`	`"set before the underlying CEF browser is created.");`
`136`	`136`	`}`
`137`	`137`
`@@ -157,7 +157,7 @@ public IRequestContext RequestContext`
`157`	`157`	`{`
`158`	`158`	`if (browserCreated)`
`159`	`159`	`{`
`160`		`- throw new Exception("Browser has already been created. RequestContext must be" +`
	`160`	`+ throw new Exception("Browser has already been created. RequestContext must be " +`
`161`	`161`	`"set before the underlying CEF browser is created.");`
`162`	`162`	`}`
`163`	`163`	`if (value != null && value.GetType() != typeof(RequestContext))`
`@@ -2299,7 +2299,7 @@ public void RegisterJsObject(string name, object objectToBind, BindingOptions op`
`2299`	`2299`
`2300`	`2300`	`if (InternalIsBrowserInitialized())`
`2301`	`2301`	`{`
`2302`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`2302`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`2303`	`2303`	`"called before the underlying CEF browser is created.");`
`2304`	`2304`	`}`
`2305`	`2305`
`@@ -2340,7 +2340,7 @@ public void RegisterAsyncJsObject(string name, object objectToBind, BindingOptio`
`2340`	`2340`
`2341`	`2341`	`if (InternalIsBrowserInitialized())`
`2342`	`2342`	`{`
`2343`		`- throw new Exception("Browser is already initialized. RegisterJsObject must be" +`
	`2343`	`+ throw new Exception("Browser is already initialized. RegisterJsObject must be " +`
`2344`	`2344`	`"called before the underlying CEF browser is created.");`
`2345`	`2345`	`}`
`2346`	`2346`	`var objectRepository = managedCefBrowserAdapter.JavascriptObjectRepository;`