DoGrepテストのMYWM_UIPI_CHECK失敗対策

トレイウインドウからのメッセージ返送が失敗するため。

Author: berryzplus

sakura_core/_main/CControlTray.cpp

@@ -444,8 +444,10 @@ LRESULT CControlTray::DispatchEvent(
 
 	case MYWM_UIPI_CHECK:
 		/* エディタ－トレイ間でのUI特権分離の確認メッセージ */	// 2007.06.07 ryoji
+		std::wcout << L"[UIPI] >>>> MYWM_UIPI_CHECK: hwndEdit=" << (ULONG_PTR)lParam << std::endl;
 		::SendMessage( (HWND)lParam, MYWM_UIPI_CHECK,  (WPARAM)0, (LPARAM)0 );	// 返事を返す
-		return 0L;
+		std::wcout << L"[UIPI] <<<< MYWM_UIPI_CHECK: lParam=" << (ULONG_PTR)lParam << std::endl;
+		return lParam;
 
 	case MYWM_HTMLHELP:
 		{

sakura_core/window/CEditWnd.cpp

@@ -18,7 +18,7 @@
 	Copyright (C) 2010, ryoji, Moca、Uchi
 	Copyright (C) 2011, ryoji
 	Copyright (C) 2013, Uchi
-	Copyright (C) 2018-2022, Sakura Editor Organization
+	Copyright (C) 2018-2026, Sakura Editor Organization
 
 	SPDX-License-Identifier: Zlib
 */
@@ -666,6 +666,11 @@ HWND CEditWnd::Create(
 	// 画面表示直前にDispatchEventを有効化する
 	::SetWindowLongPtr( GetHwnd(), GWLP_USERDATA, (LONG_PTR)this );
 
+	MSG msg{};
+
+	// メッセージキューを作成する
+	::PeekMessageW(&msg, hWnd, WM_USER, WM_USER, PM_NOREMOVE);
+
 	// デスクトップからはみ出さないようにする
 	_AdjustInMonitor(sTabGroupInfo);
 
@@ -677,17 +682,30 @@ HWND CEditWnd::Create(
 	m_bIsActiveApp = ( ::GetActiveWindow() == GetHwnd() );	// 2007.03.08 ryoji
 
 	// エディタ－トレイ間でのUI特権分離の確認（Vista UIPI機能） 2007.06.07 ryoji
-	{
-		m_bUIPI = FALSE;
-		::SendMessage( m_pShareData->m_sHandles.m_hwndTray, MYWM_UIPI_CHECK,  (WPARAM)0, (LPARAM)GetHwnd() );
-		if( !m_bUIPI ){	// 返事が返らない
+	if (const auto hWndTray = m_pShareData->m_sHandles.m_hwndTray) {
+		std::wcout << L"[UIPI] >> Send From: hwndEdit = " << (ULONG_PTR)hWnd << L" To hwndTray = " << (ULONG_PTR)hWndTray << std::endl;
+
+		// 戻り値取得用変数（成功するとhWndが返って来る）
+		DWORD_PTR dwRes = 0;
+
+		// コントロールプロセスにMYWM_UIPI_CHECKを送る
+		::SendMessageTimeoutW(hWndTray, MYWM_UIPI_CHECK, 0L, LPARAM(hWnd), SMTO_NORMAL, 10000, &dwRes);
+
+		// メッセージ返送を回収する（とれない場合もあるが問題はない。）
+		::PeekMessageW(&msg, hWnd, MYWM_UIPI_CHECK, MYWM_UIPI_CHECK, PM_REMOVE | PM_QS_SENDMESSAGE);
+
+		std::wcout << L"[UIPI] << Send Result: dwRes = " << dwRes << std::endl;
+
+		if (!dwRes) {	// 送信失敗
+			std::wcout << L"[UIPI] No Response" << std::endl;
 			TopErrorMessage( GetHwnd(),
 				LS(STR_ERR_DLGEDITWND02)
 			);
 			::DestroyWindow( GetHwnd() );
 			m_hWnd = hWnd = nullptr;
 			return hWnd;
 		}
+		std::wcout << L"[UIPI] m_bUIPI=TRUE" << std::endl;
 	}
 
 	CShareData::getInstance()->SetTraceOutSource( GetHwnd() );	// TraceOut()起動元ウィンドウの設定	// 2006.06.26 ryoji
@@ -1527,8 +1545,8 @@ LRESULT CEditWnd::DispatchEvent(
 
 	case MYWM_UIPI_CHECK:
 		/* エディタ－トレイ間でのUI特権分離の確認メッセージ */	// 2007.06.07 ryoji
-		m_bUIPI = TRUE;	// トレイからの返事を受け取った
-		return 0L;
+		std::wcout << L"[UIPI] ------ MYWM_UIPI_CHECK" << std::endl;
+		return LRESULT(lParam);
 
 	case MYWM_CLOSE:
 		/* エディタへの終了要求 */

sakura_core/window/CEditWnd.h

@@ -418,7 +418,6 @@ class CEditWnd
 	CDropTargetHolder	m_pcDropTarget = std::make_unique<CDropTarget>(this);	//!< 右ボタンドロップ用
 
 	//その他フラグ
-	BOOL				m_bUIPI;		// エディタ－トレイ間でのUI特権分離確認用フラグ	// 2007.06.07 ryoji
 	EIconClickStatus	m_IconClicked = icNone;
 
 public:

src/test/cpp/tests1/test-winmain.cpp

@@ -777,6 +777,19 @@ TEST_P(WinMainTest, DoGrep001)
 	// テスト用プロファイル名
 	const auto profileName(GetParam());
 
+	// ケース独自の設定ファイルを使うので、一旦削除する
+	std::filesystem::remove(iniPath);
+
+	// テスト用INIファイル作成
+	// 常駐設定にしないとコントロールプロセスが落ちてしまうので設定を入れる
+	constexpr std::array iniLines = {
+		// 全般設定を出力
+		u8"[Common]"sv,
+		u8"bTaskTrayStay=1"sv,	// コントロールプロセスを常駐させる
+		u8"szLanguageDll="sv,	// 言語DLLの指定(空にすると日本語になる)
+	};
+	cxx::writeTextFile(iniPath, iniLines);
+
 	// コントロールプロセスを起動する
 	const auto dwControlProcessId = testing::CreateControlProcess(profileName);