コンテキストメニュー: 「新しいタブを画像で開く」などが音声や動画にも表示される。

[HashidaTKS]

Describe the bug

コンテキストメニューの「新しいタブを画像で開く」などが音声や動画を右クリックした際にも表示される。
音声や動画は画像ではないので、それに合わせた文言であるべき。

To Reproduce

https://developer.mozilla.org/ja/docs/Web/HTML/Reference/Elements/audio

を開き、html組み込みの音声を表示する。

音声部分を右クリックしコンテキストメニューを表示する。

この時、以下のように画像向けのコンテキストメニューが表示される。

Expected behavior

「新しいタブを音声で開く」「名前を付けて音声を保存」といった、適切な名前になるべき。

Your Environment

- Chronos version: 15.1.135.0 (予定)
- ThinApps version: Any

Your Configuration

N/A

Your TRACE log (Chronos_trace.log)

N/A

Additional context

https://github.com/ThinBridge/Chronos/blob/472195d2a9e29bb834bc683af0219a57f3f1fac1/client_handler.cpp?plain=1#L396C1-L396C61

ここで if (Flg & (CM_TYPEFLAG_MEDIA | CM_MEDIATYPE_IMAGE)) != 0 をしているのだが、CM_MEDIATYPE_IMAGEは cef_context_menu_media_type_t型で、Flgの型であるcef_context_menu_type_flags_t型ではないので、この比較が間違えている。

cef_context_menu_media_type_t mediaType = params->GetMediaType();
if (mediaType == CM_MEDIATYPE_IMAGE)
{
  ...
}

のように、別途GetMediaTypeを実行して確認する必要がある。
ただし、単に上記のように修正すると音声や動画を新しいタブで開くといったことができなくなってしまうので、何らかの対応が必要。

[yashirot]

「新しいタブで画像を開く」の文言をコンテンツの種類に依存しないものに変更するのが
簡単かなという気がしました。

案として

• コンテンツを開く
• メディアを開く

とか

[HashidaTKS]

例えばyoutubeでは、動画再生中に「新しいタブで画像を開く」を選択しても開けなくてエラー画面に遷移するので、そのあたりも考慮する必要がありそうです。

• 画像 -> メディア/コンテンツ に変更
• 動かない場合は表示しない

の二つの対応が必要かな、と思います。

[HashidaTKS]

このあたり、Chronosが独自にコンテキストメニューの追加及び動作の実装をしているが、CEFのデフォルトのコンテキストメニューを使用するようにしても大丈夫な気がする。