Fix bug and add more tests

Author: amaitland

CefSharp.BrowserSubprocess.Core/CefAppUnmanagedWrapper.cpp

@@ -180,7 +180,7 @@ namespace CefSharp
                         {
                             auto origin = _jsBindingApiAllowOrigins->GetString(i);
 
-                            if (origin.compare(frameUrlOrigin))
+                            if (origin.compare(frameUrlOrigin) == 0)
                             {
                                 createObjects = true;
 

CefSharp.Test/JavascriptBinding/JavascriptBindingTests.cs

@@ -157,14 +157,16 @@ public async Task ShouldDisableJsBindingApiForOrigin()
             }
         }
 
-        [Fact]
-        public async Task ShouldEnableJsBindingApiForOrigin()
+        [Theory]
+        [InlineData(CefExample.ExampleDomain)]
+        [InlineData("someorigin", CefExample.ExampleDomain)]
+        public async Task ShouldEnableJsBindingApiForOrigin(params string[] origins)
         {
             using (var browser = new ChromiumWebBrowser(CefExample.BindingApiCustomObjectNameTestUrl, automaticallyCreateBrowser: false))
             {
                 var settings = browser.JavascriptObjectRepository.Settings;
                 settings.JavascriptBindingApiEnabled = true;
-                settings.JavascriptBindingApiAllowOrigins = new string[] { CefExample.ExampleDomain };
+                settings.JavascriptBindingApiAllowOrigins = origins;
 
                 //To modify the settings we need to defer browser creation slightly
                 browser.CreateBrowser();