Skip to content

UIプロセス分離の確認仕様を変更する#2455

Merged
berryzplus merged 2 commits into
sakura-editor:masterfrom
berryzplus:feature/change_uipi_check
Apr 29, 2026
Merged

UIプロセス分離の確認仕様を変更する#2455
berryzplus merged 2 commits into
sakura-editor:masterfrom
berryzplus:feature/change_uipi_check

Conversation

@berryzplus

Copy link
Copy Markdown
Contributor

PR対象

  • アプリ(サクラエディタ本体)
  • テストコード

カテゴリ

  • 削除

PR の背景

  • 先行 issue はとくに作成していません。

WinMainTest::DoGrep001が常に失敗する対策を行います。

仕様・動作説明

  • UIプロセス分離対策のチェック仕様を変更します。
Edit→Tray Tray→Edit
変更前 送るだけ 送り返す
変更後 戻り値も見る (廃止)

本件、勝手にやります。

PR の影響範囲

  • DoGrep001 テストが通るようになります。
  • 誤って「管理者として実行」した場合の振る舞いに、影響が出る可能性があります。

テスト内容

関連 issue, PR

参考資料

@github-actions

github-actions Bot commented Apr 29, 2026

Copy link
Copy Markdown

Test Results

1 098 tests  ±0   1 098 ✅ ±0   3m 54s ⏱️ - 2m 44s
  103 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit bd3be55. ± Comparison against base commit 4cd960b.

♻️ This comment has been updated with latest results.

@berryzplus

Copy link
Copy Markdown
Contributor Author

正常終了するケースのログは以下の通り。
image

異常終了するDoGrep001のログは以下の通り。
image

想定と異なるので一旦クローズ。

@berryzplus berryzplus closed this Apr 29, 2026
トレイウインドウからのメッセージ返送が失敗するため。
@berryzplus

Copy link
Copy Markdown
Contributor Author

対策見つかったので re-open できるかの検証を兼ねて再オープンします。

(sourceブランチが pull-request の最終状態と一致していれば re-open 可能っぽい。)

@berryzplus berryzplus reopened this Apr 29, 2026
@berryzplus berryzplus force-pushed the feature/change_uipi_check branch from c64465b to 00e1e34 Compare April 29, 2026 03:16
@berryzplus

Copy link
Copy Markdown
Contributor Author

正常終了するテストケースのログ。
image

異常終了するテストケースのログ。(CEditWndによる返送ログが出ない。)
image

vcpkgが使えないのでspdlogを取り込めず、C++のログを残しておくわけにいかないため。
@sonarqubecloud

Copy link
Copy Markdown

@berryzplus berryzplus marked this pull request as ready for review April 29, 2026 04:23
@berryzplus berryzplus merged commit 71514c3 into sakura-editor:master Apr 29, 2026
13 checks passed
@berryzplus berryzplus deleted the feature/change_uipi_check branch April 29, 2026 08:20
hpmy-dev pushed a commit to hpmy-dev/sakura that referenced this pull request May 15, 2026
…uipi_check

UIプロセス分離の確認仕様を変更する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant