Merge pull request #2169 from github/fixes/vs2015-web-browser

grokys · web-flow · commit 11c9d3451380 · 2019-01-10T12:34:14.000+01:00
Fix broken Browser service.
diff --git a/src/GitHub.VisualStudio/Helpers/Browser.cs b/src/GitHub.VisualStudio/Helpers/Browser.cs
@@ -44,7 +44,7 @@ public void OpenUrl(Uri uri)
                 int result = ErrorHandler.CallWithCOMConvention(() =>
                 {
                     ThreadHelper.ThrowIfNotOnUIThread();
-                    service.CreateExternalWebBrowser((uint)createFlags, resolution, uri.AbsoluteUri);
+                    return service.CreateExternalWebBrowser((uint)createFlags, resolution, uri.AbsoluteUri);
                 });
 
                 if (ErrorHandler.Succeeded(result))

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ public void OpenUrl(Uri uri)`
`44`	`44`	`int result = ErrorHandler.CallWithCOMConvention(() =>`
`45`	`45`	`{`
`46`	`46`	`ThreadHelper.ThrowIfNotOnUIThread();`
`47`		`- service.CreateExternalWebBrowser((uint)createFlags, resolution, uri.AbsoluteUri);`
	`47`	`+ return service.CreateExternalWebBrowser((uint)createFlags, resolution, uri.AbsoluteUri);`
`48`	`48`	`});`
`49`	`49`
`50`	`50`	`if (ErrorHandler.Succeeded(result))`