From b6a686db2855acb5b5a8bcfa7a04b83aa92672ab Mon Sep 17 00:00:00 2001
From: JKlingen <jens@getgreenshot.org>
Date: Fri, 14 Sep 2012 20:55:42 +0000
Subject: [PATCH] two fixes to avoid exceptions when trying to capture IE
 content from IE instances embedded in other applications

git-svn-id: http://svn.code.sf.net/p/greenshot/code/trunk@2043 7dccd23d-a4a3-4e1f-8c07-b4c1b4018ab4
---
 Greenshot/Helpers/IECaptureHelper.cs | 54 +++++++++++++++-------------
 1 file changed, 30 insertions(+), 24 deletions(-)

diff --git a/Greenshot/Helpers/IECaptureHelper.cs b/Greenshot/Helpers/IECaptureHelper.cs
index 1e884217a..058464606 100644
--- a/Greenshot/Helpers/IECaptureHelper.cs
+++ b/Greenshot/Helpers/IECaptureHelper.cs
@@ -48,12 +48,14 @@ namespace Greenshot.Helpers {
 		// Helper method to activate a certain IE Tab
 		public static void ActivateIETab(WindowDetails ieWindowDetails, int tabIndex) {
 			WindowDetails directUIWindowDetails = IEHelper.GetDirectUI(ieWindowDetails);
-			// Bring window to the front
-			ieWindowDetails.Restore();
-			// Get accessible
-			Accessible ieAccessible = new Accessible(directUIWindowDetails.Handle);
-			// Activate Tab
-			ieAccessible.ActivateIETab(tabIndex);
+			if(directUIWindowDetails != null) {
+				// Bring window to the front
+				ieWindowDetails.Restore();
+				// Get accessible
+				Accessible ieAccessible = new Accessible(directUIWindowDetails.Handle);
+				// Activate Tab
+				ieAccessible.ActivateIETab(tabIndex);
+			}
 		}
 
 		/// <summary>
@@ -411,25 +413,29 @@ namespace Greenshot.Helpers {
 
 				// Store the URL of the page
 				if (documentContainer.Url != null) {
-					Uri uri = new Uri(documentContainer.Url);
-					capture.CaptureDetails.AddMetaData("URL", uri.OriginalString);
-					// As the URL can hardly be used in a filename, the following can be used
-					if (!string.IsNullOrEmpty(uri.Scheme)) {
-						capture.CaptureDetails.AddMetaData("URL_SCHEME", uri.Scheme);
+					try {
+						Uri uri = new Uri(documentContainer.Url);
+						capture.CaptureDetails.AddMetaData("URL", uri.OriginalString);
+						// As the URL can hardly be used in a filename, the following can be used
+						if (!string.IsNullOrEmpty(uri.Scheme)) {
+							capture.CaptureDetails.AddMetaData("URL_SCHEME", uri.Scheme);
+						}
+						if (!string.IsNullOrEmpty(uri.DnsSafeHost)) {
+							capture.CaptureDetails.AddMetaData("URL_HOSTNAME", uri.DnsSafeHost);
+						}
+						if (!string.IsNullOrEmpty(uri.AbsolutePath)) {
+							capture.CaptureDetails.AddMetaData("URL_PATH", uri.AbsolutePath);
+						}
+						if (!string.IsNullOrEmpty(uri.Query)) {
+							capture.CaptureDetails.AddMetaData("URL_QUERY", uri.Query);
+						}
+						if (!string.IsNullOrEmpty(uri.UserInfo)) {
+							capture.CaptureDetails.AddMetaData("URL_USER", uri.UserInfo);
+						}
+						capture.CaptureDetails.AddMetaData("URL_PORT", uri.Port.ToString());
+					} catch(Exception e) {
+						LOG.Warn("Exception when trying to use url in metadata "+documentContainer.Url,e);
 					}
-					if (!string.IsNullOrEmpty(uri.DnsSafeHost)) {
-						capture.CaptureDetails.AddMetaData("URL_HOSTNAME", uri.DnsSafeHost);
-					}
-					if (!string.IsNullOrEmpty(uri.AbsolutePath)) {
-						capture.CaptureDetails.AddMetaData("URL_PATH", uri.AbsolutePath);
-					}
-					if (!string.IsNullOrEmpty(uri.Query)) {
-						capture.CaptureDetails.AddMetaData("URL_QUERY", uri.Query);
-					}
-					if (!string.IsNullOrEmpty(uri.UserInfo)) {
-						capture.CaptureDetails.AddMetaData("URL_USER", uri.UserInfo);
-					}
-					capture.CaptureDetails.AddMetaData("URL_PORT", uri.Port.ToString());
 				}
 
 				// Only move the mouse to correct for the capture offset