UniWinApi.IsActiveではウィンドウの存在チェックも追加。

エディタでWindowController.pickedColorがリードオンリーとなるようにした。

Author: kirurobo

Assets/UniWinApi/Scripts/Editor/WindowControllerEditor.cs

@@ -60,6 +60,17 @@ public override void OnInspectorGUI()
 	}
 }
 
+[CustomPropertyDrawer(typeof(ReadOnlyAttribute))]
+public class WindowControllerReadOnlyDrawer : PropertyDrawer
+{
+	public override void OnGUI(Rect position, SerializedProperty property, GUIContent label)
+	{
+		GUI.enabled = false;
+		EditorGUI.PropertyField(position, property, label, true);
+		GUI.enabled = true;
+	}
+}
+
 /// <summary>
 /// Set to readonly during playing
 /// Reference: http://ponkotsu-hiyorin.hateblo.jp/entry/2015/10/20/003042

Assets/UniWinApi/Scripts/UniWinApi.cs

@@ -75,7 +75,7 @@ public override string ToString()
 	/// ウィンドウ操作ができる状態ならtrueを返す
 	/// </summary>
 	/// <value><c>true</c> if this instance is active; otherwise, <c>false</c>.</value>
-	public bool IsActive { get{return hWnd != IntPtr.Zero;} }
+	public bool IsActive { get{return (hWnd != IntPtr.Zero && WinApi.IsWindow(hWnd));} }
 
 	/// <summary>
 	/// ウィンドウが最大化されていればtrue
@@ -664,6 +664,12 @@ private static IntPtr MessageCallback(int nCode, IntPtr wParam, ref WinApi.MSG l
 	/// </summary>
 	public void BeginFileDrop()
 	{
+		if (!IsActive)
+		{
+			EndFileDrop();
+			return;
+		}
+
 		if (!instances.Contains(this)) {
 			instances.Add(this);
 		}
@@ -684,26 +690,31 @@ public void EndFileDrop()
 		EndHook();
 		instances.Remove(this);
 
+		if (!IsActive) return;
+
+		// ドロップを受け付ける状態を元に戻す
 		long exstyle = this.CurrentWindowExStyle;
-		exstyle &= ~WinApi.WS_EX_ACCEPTFILES;	// ドロップ受付をやめる
+		exstyle &= ~WinApi.WS_EX_ACCEPTFILES;
 		exstyle |= (WinApi.WS_EX_ACCEPTFILES & this.OriginalWindowExStyle);	// 元からドロップ許可ならやはり受付
 		this.CurrentWindowExStyle = exstyle;
 		WinApi.SetWindowLong(hWnd, WinApi.GWL_EXSTYLE, this.CurrentWindowExStyle);
-
 	}
 
 	/// <summary>
 	/// メッセージフックを開始
 	/// </summary>
 	private void BeginHook()
 	{
-		if (IsHookSet) return;
+		// ウィンドウが操作不可なら設定はできない
 		if (!IsActive)
 		{
-			IsHookSet = false;
+			EndHook();	// フック設定なしということにしておく
 			return;
 		}
 
+		// フック設定済みなら二重には設定しない
+		if (IsHookSet) return;
+
 		// フックを設定
 		uint threadId = WinApi.GetCurrentThreadId();
 		IntPtr module = WinApi.GetModuleHandle(null);
@@ -718,7 +729,6 @@ private void BeginHook()
 		}
 
 		IsHookSet = true;
-
 		//Debug.Log("BeginHook");
 	}
 
@@ -733,9 +743,6 @@ private void EndHook()
 			myHook = IntPtr.Zero;
 			myHookCallback = null;
 
-			// Unityだと通常はドラッグ不可のはずなので戻しても良いのかも
-			WinApi.DragAcceptFiles(hWnd, false);
-
 			//Debug.Log("EndHook");
 		}
 		IsHookSet = false;
@@ -746,7 +753,7 @@ private void EndHook()
 	/// </summary>
 	public void Dispose()
 	{
-		if (hWnd != IntPtr.Zero && WinApi.IsWindow(hWnd))
+		if (IsActive)
 		{
 			EndFileDrop();
 #if UNITY_EDITOR

Assets/UniWinApi/Scripts/WindowController.cs

@@ -9,12 +9,17 @@
 using UnityEngine;
 
 /// <summary>
-/// Set to readonly during playing
+/// Set editable the bool property
 /// </summary>
 [System.AttributeUsage(System.AttributeTargets.Field, Inherited = true, AllowMultiple = false)]
-public class BoolPropertyAttribute : PropertyAttribute
-{
-}
+public class BoolPropertyAttribute : PropertyAttribute {}
+
+/// <summary>
+/// Set the attribute as readonly
+/// </summary>
+[System.AttributeUsage(System.AttributeTargets.Field, Inherited = true, AllowMultiple = false)]
+public class ReadOnlyAttribute : PropertyAttribute { }
+
 
 /// <summary>
 /// デスクトップマスコット風の利用法を想定した UniWinApi サンプル。
@@ -113,7 +118,7 @@ public bool enableFileDrop {
 	/// <summary>
 	/// Pixel color under the mouse pointer. (Read only)
 	/// </summary>
-	[Tooltip("Pixel color under the mouse pointer. (Read only)")]
+	[ReadOnly, Tooltip("Pixel color under the mouse pointer. (Read only)")]
 	public Color pickedColor;
 
 	private bool isDragging = false;