ThinBridge / Chronos

Chromiumベースの業務ブラウザ
Other
5 stars 3 forks source link

[CEF125+] DevTools doesn't pop up. #203

Open HashidaTKS opened 2 weeks ago

HashidaTKS commented 2 weeks ago

On Chronos with CEF125+, DevTools doesn't pop up with new window but attaches to existing window,

FYI: https://www.magpcss.org/ceforum/viewtopic.php?f=6&t=19792

HashidaTKS commented 1 week ago

https://www.magpcss.org/ceforum/viewtopic.php?f=6&t=19792

の通り

void ClientHandler::OnBeforeDevToolsPopup(CefRefPtr<CefBrowser> browser,
                    CefWindowInfo& windowInfo,
                    CefRefPtr<CefClient>& client,
                    CefBrowserSettings& settings,
                    CefRefPtr<CefDictionaryValue>& extra_info,
                    bool* use_default_window) {
    windowInfo.SetAsWindowless(0);
}

を追加するとDevToolsがポップアップするようになる。 ただし、DevToolsのアイコンがChromiumになってしまう。

image

HashidaTKS commented 1 week ago

この方針で、OnBeforeDevToolsPopupで何か追加の設定をすることで回避できないか試していたが、今のところ解決策を発見できていない。

HashidaTKS commented 1 week ago

https://github.com/chromiumembedded/cef/issues/3685

DevTools popups are Chrome style only (cannot be windowless).

うーん。。。

HashidaTKS commented 1 week ago

DevToolを使う機会はそうそう多くないことと、モダンなブラウザの仕様に合わせて、現在のタブの表示にドッキングされても特に不都合はない気がする。 その方針の仕様変更とするのはどうか?

HashidaTKS commented 1 week ago

現在のタブにドッキングして表示するようにする場合も、現在は正しく動いていないので(DevToolsをリサイズできない等)、そのあたりの修正は必要。 8h以内で調査して対応が無理という感触なら、 https://github.com/ThinBridge/Chronos/issues/203#issuecomment-2185753810 の方針で対応して、アイコンが変更できるかどうか、できなければそのままでよいか検討するのはどうか。